-
Notifications
You must be signed in to change notification settings - Fork 2
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
HTMLのソース上でhrefの値が切り替わらない問題を修正しました #83
Conversation
…on the saved settings in the admin panel
@kurudrive @drill-lancer @mthaichi すみません、テストが以下のようになり通りません。
どうやら docker-compose が見つからないために発生しているようです。 |
@kurudrive |
@mtdkei ありがとうございます! |・w・).oO( readme.txt 書いてなかったでー ) |
@kurudrive |
調整ありがとうございます!確認しました。 |
あ、ちょっと気になるところがありましたので、お昼後にまた少し確認します🙇 |
「File Link」ボタンからファイルを選択するとurlが入らないかもと思い見ていたのですが(masterブランチの |
チケットへのリンク / 変更の理由(元のissueがあればリンクを貼り付ければOK)
#79
どういう変更をしたか?
JSにてdecodeURIComponentを使用して、エンコードされたURLをデコードしてから比較するようにしました。これにより、エンコードされたURLが正しく一致し、対象のリンク要素が正確に特定されるようになりました。
また、PHPではリンク (vk-ltc-link) が設定された投稿をすべて対象とし、そのリンクを正しいリダイレクトURLに置き換えるようにしたことで、管理画面で指定されたリダイレクト先のURLが正しく反映されるようになりました。
変更前 Before
変更後 After
レビューに回す前に確認する事
プログラムの変更の場合
変更内容について何を確認したか、どういう方法で確認をしたかなど
Lightning、TT4、Nishikiで以下を確認しました。
また、「別ウィンドウで開く」の有無での挙動を確認しました。
確認URL
( どこかのデモサイトかテストサーバーにデプロイ済みなどで確認できる場合はそのURL )
レビュワーの確認方法・確認する内容など
レビュワーに回す前の確認事項
レビュワー向け
確認して変更が反映されていない場合の確認事項