Release 2024-07-03 06:36:16 +0000 #7920
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
github-actions
bot
commented
Jul 2, 2024
•
github-actions
bot
commented
- 特別イベント一覧、定期イベント一覧に、直近(近日開催)のイベント一覧を表示する。 #7272 @masyuko0222
- プラクティス個別ページ配下のDocsが0の場合に泣き顔を表示するようにした #7816 @ham-cap
- 管理者は自分以外のユーザーのGitHubアカウント連携を解除できるように修正 #7819 @Shrimprin
- 管理者が他のメンターのプロフィール公開・非公開を変更できるように修正 #7857 @Shrimprin
- 提出物の新規作成/編集画面にキャンセルボタンを追加 #7864 @motohiro-mm
- meta description中にプラクティス名が表示されるように修正 #7880 @su-su-su-su
・このissueの変更が反映されると、プラクティスに紐づく日報、質問、Docsのうち質問だけが0件の時でも笑顔マークになってしまうため、町田さんに確認のうえ、全て泣き顔で統一することとした。
・町田さんからの指示を反映。日報のページのみ泣き顔マークの背景が白くなっていたため、他のページと統一するために白背景を削除した。
コンフリクト解消のためrebaseしたところ「日報はまだありません。」の表示が重複するようになってしまったため修正した。
他のキャンセルボタン（日報、イベント）と位置を合わせるため、`.is-ais-flex-start`を削除した
- テストが重くなるため、動作確認用のfixtureを削除し、既存のfixtureにgithub_idを追加
Dateクラスを利用すると検索時にUTCに変換してくれずバグになるため。
selectだと、SQLのselectが期待されてしまうため、勘違いさせないように。
UpcomingEventsGroupクラスを作成していたが、近日開催イベントであるUpcomingEvent自体がその作成を担っても違和感はないため。
Structだと内部でしか使わないことを想定されるが、ContorllerでViewに渡すためにStructを使ってしまってる感じだったので変更した。 クラスもあまり増やしたくなかったので、シンプルにハッシュに変えた。 あと利用していないkeyが潜んでいたのでそれも消した。
・クラスとして定義することで、どのような構造かすぐにわかるようにした。 ・また、副作用がないので、利用者からはUpcomingEventのメンバに見えるようにした。 - 参考：https://docs.komagata.org/5630
・UpcomingEventの責務として、外部に公開しておくべきメソッドのため。 ・.upcoming_events_groupsの内部で利用されているため、テストはそちらで担保する。 - テストコード見やすくするために、コード順も少し変えた。
…users-github 管理者は自分以外のユーザーのGitHubアカウント連携を解除できるように修正
github-actions
bot
changed the title
…n_special_and_regular_events_pages 特別イベント一覧、定期イベント一覧に、直近(近日開催)のイベント一覧を表示する。
github-actions
bot
changed the title
…roduct-edit-page 提出物の新規作成/編集画面にキャンセルボタンを追加
github-actions
bot
changed the title
…eta-description meta description中にプラクティス名が表示されるように修正
github-actions
bot
changed the title
…docs-page-in-an-individual-practice-page-when-no-docs プラクティス個別ページ配下のDocsが0の場合に泣き顔を表示するようにした
github-actions
bot
changed the title
…ile-not-saving 管理者が他のメンターのプロフィール公開・非公開を変更できるように修正
github-actions
bot
changed the title
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.