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

anchorと関連jsが動いてなかったので修正対応 #24

Merged
merged 1 commit into from
Mar 6, 2024

Conversation

moonstruckdrops
Copy link

元の実装は、このようにjsでpostして動かすようになっているらしい

しかしながら、現状の実装だとリンクのままgetでアクセスするようになっているので、エラーになるようです
そのため、元の実装に追従してPOSTで動かすようにしました
変更元のこちらをみる限り、JS側は対応できているようなので画面の内容だけ修正しています
b8f32ca#diff-5b2f7bbe8fedd91a0c470a11e3e8dfdf75eceafa4be0f9c83b87324de2173562

とはいえ、これまで言われてこなかったのであんまり気にするような機能でもないらしい

Copy link

@yoyogix yoyogix left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ありがとうございます〜!
2.5系へのupdate時に併せて本番反映しようと思います。

@yoyogix
Copy link

yoyogix commented Feb 7, 2024

ありがとうございます〜! 2.5系へのupdate時に併せて本番反映しようと思います。

前言撤回
2.5系へのupdate対応の時期も決まっていないので、先に本番反映していきます〜

@shigerix
Copy link

shigerix commented Mar 6, 2024

staging 触ってあれ?404だって思ってたらマージしてなかった・・・
マージします!

@shigerix shigerix merged commit a96ce5b into staging-smarthr Mar 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants