Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

モバイル(ガラケー)サイトの廃止 #194

Closed
nanasess opened this issue Apr 27, 2018 · 8 comments
Closed

モバイル(ガラケー)サイトの廃止 #194

nanasess opened this issue Apr 27, 2018 · 8 comments

Comments

@nanasess
Copy link
Contributor

  • もうガラケーで https にアクセスできない時代になってしまったので、この際削除してしまってよいと思われる
  • Net_UserAgent_Mobile/Net/UserAgent/Mobile/DoCoMo/ScreenInfo.php の 2088行目に、無効な参照代入があるため、ガラケーを活かすのであれば、 composer を使わずローカルに持ってくる必要がある
  • 最低限の下位互換性のため、 既存のクラスや定数は残しておく
    • SC_MobileUserAgent のメソッドは常に false or null を返す
@nanasess
Copy link
Contributor Author

@so-amuamu
Copy link

動作無効にして、コード類は置いておく。
表向きはNot supportにしましょう

@so-amuamu
Copy link

管理画面系も隠さないと駄目ですかね?

@nanasess
Copy link
Contributor Author

管理画面はそのままでいいような

@nobuhiko
Copy link
Contributor

ナビだけコメントアウトするぐらいならすぐ出来ますが

@Yangsin
Copy link

Yangsin commented Aug 2, 2018

いまのところ不要という意見しかないようですので、
メンテナー6名の過半数の同意 or 8月6日(月)までに反対意見がないなら
@so-amuamu のご提案いただいた以下ので内容でDisscussion終えましょう。

動作無効にして、コード類は置いておく。
表向きはNot supportにしましょう

@so-amuamu
Copy link

ショップ運営者が迷う要素になるので、ナビのコメントアウトだけはしましょうかね

ナビだけコメントアウトするぐらいならすぐ出来ますが

chihiro-adachi pushed a commit that referenced this issue Aug 3, 2018
fix #194 ガラケーを無効にする
@Yangsin
Copy link

Yangsin commented Aug 3, 2018

誤って本日マージされてますが、反対意見などがあれば戻すということで。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants