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

スキル削除機能を追加 #549

Merged
merged 8 commits into from
Dec 9, 2024
Merged

Conversation

NePonpon
Copy link
Member

@NePonpon NePonpon commented Dec 5, 2024

チケットURL

GH-548

対応内容・対応背景・妥協点

スキル削除を追加
スキル設定画面にスキル削除ボタンを追加
image

やったこと

  • functions -> function
  • アイテムのコンポーネント化

やってないこと

makeup:skill/trigger/error/mismatched_triggerからスキル削除のボタンを削除する -> GH-547

チェックリスト (ガイドライン)

  • PR
    • PRのタイトルはわかり易い名前が設定されていますか?(日本語でも可)
    • PRの必須項目はすべて記載していますか?
    • PRの必要ない項目は削除していますか?
    • PRの内容と変更内容は一致していますか?
      • 1機能=1PRであるべきです。1機能の途中でも問題ありません
  • Commit
    • Commitメッセージはルール通りですか?
      • コミットメッセージの先頭に[Add|Delete|Modify|Fix|Refactor|Move]等の動詞の原形を追加してください。(参考) (参考2)
      • コミットメッセージの説明にはGH-〇〇でチケット番号をつけてください
        (チケットが存在しない場合はNO-ISSUEにしてください
      • マージコミットの場合はこの限りではありません
    • コミットの粒度は適切ですか?
      • 1コミット=1要素(1ロジック)の変更であるべきです
  • Branch
    • ブランチの名前は正しいですか?
      • ブランチ名先頭はfeature/[簡単な説明]fix/[簡単な説明] の何れかであるべきです
      • 簡単な説明は英語であるべきです(不具合発生するので)
    • ブランチの向き、切り先は正しいですか?

@NePonpon NePonpon self-assigned this Dec 5, 2024
@NePonpon NePonpon requested a review from a team as a code owner December 5, 2024 11:18
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Code review by ChatGPT

@NePonpon NePonpon linked an issue Dec 5, 2024 that may be closed by this pull request
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Code review by ChatGPT

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Code review by ChatGPT

@yabane yabane linked an issue Dec 9, 2024 that may be closed by this pull request
@NePonpon NePonpon merged commit e4ebdea into 1-21-1-dev Dec 9, 2024
2 checks passed
@NePonpon NePonpon deleted the feature/skill-trigger-delete branch December 9, 2024 13:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

スキル削除 - skill:trigger/delete/ [要望] スキル削除を簡単にしたい
2 participants