-
Notifications
You must be signed in to change notification settings - Fork 71
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
イベント購読に自分が参加している定期イベントも反映させたい。 #6607
Comments
このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。 |
このissue|PRはstaleラベルを付けた後7日間更新がないためcloseしました。 |
このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。 |
@machida これってどういう機能でしょうか。もう少し情報いただければありがたいです〜。 |
このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。 |
@komagata 詳細を書きましたー |
@machida ありがとうございます~! |
このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。 |
このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。 |
本番環境での動作が確認できたのでcloseします |
@a-terumoto-gs #8119 でカレンダーにエラーが起きているようです。確認お願いできればありがたいです。` |
@komagata |
@a-terumoto-gs すみません、もう @a-terumoto-gs さんは卒業されているので作業は不要です。
もしよろしければ、こちら(旧仕様で使われていた)についてもう少しヒントをいただければありがたいです〜。 |
@komagata イベント購読をする際にはcalendars_controllerにリクエストをおくってイベントのデータを取ってくるようにしています。 以前はどのユーザーも共通でこれから開催の特別イベントのデータを取ってくる仕様だったため、 今回の仕様変更でユーザーごとにカスタムされたイベントのデータが必要になったため、リクエスト時に必要な情報としてuser_idを追加で渡すようにしております。 エラーがでているのは この説明で伝わりますでしょうか? |
@a-terumoto-gs なるほどです。 |
また再現したらやるとしてこちらは一旦Closeしたいと思います。 |
/events にアクセス。
イベント購読ボタンをクリックすると、参加・不参加関係無しに、全ての特別イベントがカレンダーに反映される。
参加・不参加関係無いので、カレンダーに反映される結果はどのユーザーも同じである。なので、どのユーザーもボタンのリンク先も同じ。
以下が、今回行って欲しい機能追加、変更。
自分が参加登録している特別イベントは、参
加登録済み、のような文言がカレンダーに表示されるようにしたい。参加登録していない特別イベントも今と変わらずカレンダーに表示されるようにする。
自分が参加登録している定期イベントもカレンダーに表示されるようにしてほしい。今までは特別イベントだけが表示されていたので、定期イベントがカレンダーに表示されるのは完全に新機能。
特別イベントは自分が参加・不参加関係なくカレンダーに表示されるが、定期イベントは自分が参加登録しているものだけが表示されるようにする。
また、定期イベントが祝日などの関係で休みになる場合は、カレンダーにも表示しないようにする。
今まではどのユーザーもイベント購読ボタンのURLは同じだったが、今回の変更によりユーザーによってカレンダーに表示される内容が異なるので、ユーザーごとにボタンのURLが異なるようになる。
The text was updated successfully, but these errors were encountered: