Skip to content

フロントエンド統合し完成させる - #100

Merged
hinshiba merged 72 commits into
mainfrom
front/82-integrate
Aug 14, 2026
Merged

フロントエンド統合し完成させる#100
hinshiba merged 72 commits into
mainfrom
front/82-integrate

Conversation

@hinshiba

Copy link
Copy Markdown
Owner

スタックPRを完了しmainへマージする

エンドポイントを外部から注入できるように変更
@sunakun1331

sunakun1331 commented Aug 14, 2026

Copy link
Copy Markdown
Collaborator

7.科目管理画面の「講義番号」入力欄は<西暦>+<6桁の番号>となるため10桁になることが確定しています,10桁以外でも登録ができてしまいます.また講義番号の最初の4桁は必ず西暦になるためそこに数字以外の文字が入力された場合その時点で間違っていることが分かります.(私の担当なのにごめんなさい)))

8.もう一度科目を登録する際は更新サイトの更新が必要である旨を表記するか,または登録ボタンを常に表示しておくのがよいと思います.

@hinshiba

Copy link
Copy Markdown
Owner Author

@sunakun1331

7.科目管理画面の「講義番号」入力欄は<西暦>+<6桁の番号>となるため10桁になることが確定しています,10桁以外でも登録ができてしまいます.また講義番号の最初の4桁は必ず西暦になるためそこに数字以外の文字が入力された場合その時点で間違っていることが分かります.(私の担当なのにごめんなさい)))

かなりナイスな指摘
ありがとう.

stkiritn
stkiritn previously approved these changes Aug 14, 2026

@stkiritn stkiritn left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

これでいいと思います.102で気になったことは書いたので,approveしときます!

@sunakun1331 sunakun1331 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

技術的な問題は見当たらないため,approveします.

@hinshiba
hinshiba dismissed coderabbitai[bot]’s stale review August 14, 2026 15:13

なぜかConversationがすべて解決されているにもかかわらずApproveしないため

@stkiritn stkiritn left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

再approve

@hinshiba
hinshiba merged commit d4996c1 into main Aug 14, 2026
2 checks passed
@hinshiba
hinshiba deleted the front/82-integrate branch August 14, 2026 15:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

frontend Something About Frontend Implementation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

フロントエンドを統合し利用可能にする

3 participants