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

feat/new role and edit role #226

Merged
merged 22 commits into from
Jan 7, 2025
Merged

feat/new role and edit role #226

merged 22 commits into from
Jan 7, 2025

Conversation

yawn-c111
Copy link
Collaborator

@yawn-c111 yawn-c111 commented Dec 21, 2024

ロールの新規作成ページと編集ページ

Description

  • ページ新規作成
    • ロールの新規作成
    • ロールの編集
  • 上記2ページに共通のコンポーネント作成
  • Dialog(モーダル)
  • ロールの権利または権限のリスト表示
  • Hatsの情報取得周りを追加
  • 既存コンポーネントにオプションの機能を追加

Fixes # (issue)

Type of change

Please delete options that are not relevant.

  • New feature (non-breaking change which adds functionality)

Screenshots

image

@yawn-c111 yawn-c111 changed the title WIP/new role and edit role feat/new role and edit role Jan 6, 2025
@yawn-c111 yawn-c111 marked this pull request as ready for review January 6, 2025 08:16
@yawn-c111
Copy link
Collaborator Author

@yu23ki14

大変お待たせいたしました 🙏
お手すきの際によろしくお願いします!

以下の2点を相談させてください!

  • 新規作成するロール(Hat)の親Hatは、全てHatterHatでよいか?
  • ロールを作成・編集した後の遷移先は、$treeId_.$roles_.index.tsx(未作成)でよいか?

@yawn-c111
Copy link
Collaborator Author

完全に重複していた部分をリファクタして一つにまとめました。

@yawn-c111
Copy link
Collaborator Author

画像入力コンポーネントの上部マージンを修正しました。

@yu23ki14
Copy link
Member

yu23ki14 commented Jan 6, 2025

@yawn-c111

  • 新規作成するロール(Hat)の親Hatは、全てHatterHatでよいか?

はい大丈夫です!

  • ロールを作成・編集した後の遷移先は、$treeId_.$roles_.index.tsx(未作成)でよいか?

はい大丈夫です!

@yu23ki14
Copy link
Member

yu23ki14 commented Jan 6, 2025

@yawn-c111
編集ページで未編集のまま保存すると、そのまま/rolesページに飛ばされる。未編集の場合は保存ボタンをDisabledにする。

https://www.awesomescreenshot.com/video/35241405?key=f253c5b79d056fba18e959a94ec3d3a2

Copy link
Member

Choose a reason for hiding this comment

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

InputDescription, InputImage, InputLink, InputNameを /components/input/~~~.tsx にディレクトリ一つ掘っていただけると!

@yawn-c111
Copy link
Collaborator Author

@yu23ki14
レビューありがとうございました!
上記2点を修正しましたので、ご確認よろしくお願いします!

@yu23ki14 yu23ki14 merged commit 2d35587 into main Jan 7, 2025
2 checks passed
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.

2 participants