feat: マナが無くなったら掘れなくするパッシブスキルを追加 #2370

kuroma6666 · 2024-08-31T12:54:30Z

close #1964

このPRの変更点と理由:

主な変更点

パッシブスキルに対して、スキルをONにしている際にマナを消費しきった際に整地スキルによってブロック破壊しない設定を追加

補足情報:

デフォルト設定値はOFF設定とする（初心者プレイヤー詰み防止）
パッシブスキル設定は永続化することとしました

kuroma6666 · 2024-09-14T17:38:33Z

@rito528
ご確認よろしくお願いします。

rito528

develop ブランチの変更を取り込んでしまっているようなので、 git rebase --rebase-merges 1_18 などを実行して develop ブランチの変更をコミット履歴から消していただけると助かります🙇‍♂️🙇‍♂️

kuroma6666 · 2024-09-15T05:27:58Z

了解です。

`breakskilltargetconfig`で扱えない事柄を扱う為のサブシステム

kuroma6666 · 2024-09-15T06:49:05Z

@rito528
developブランチの変更を取り込んでしまっていた内容については削除しました。

rito528

一旦2つ変更をお願いします

src/main/scala/com/github/unchama/seichiassist/listener/PlayerBlockBreakListener.scala

src/main/scala/com/github/unchama/seichiassist/util/BreakUtil.scala

kuroma6666 · 2024-09-21T07:48:50Z

一旦2つ変更をお願いします

@rito528
一旦変更かけました

...ssist/subsystems/breakskilltriggerconfig/application/actions/BreakSkillTriggerSettings.scala

src/main/scala/com/github/unchama/seichiassist/subsystems/mana/domain/ManaManipulation.scala

src/main/scala/com/github/unchama/seichiassist/listener/PlayerBlockBreakListener.scala

*GiganticMinecraft#2370 (comment)

kuroma6666 · 2024-10-12T07:57:07Z

テーブル構成が変わることで今と実装がかなり変わりそうなので、一度以下の内容の変更をお願いします！

@rito528
一旦変更をかけたので、確認お願いします。

...unchama/seichiassist/subsystems/breakskilltriggerconfig/domain/BreakSkillTriggerConfig.scala

...hama/seichiassist/subsystems/breakskilltriggerconfig/domain/BreakSkillTriggerConfigKey.scala

...bsystems/breakskilltriggerconfig/infrastructure/JdbcBreakSkillTriggerConfigPersistence.scala

src/main/scala/com/github/unchama/seichiassist/subsystems/mana/domain/ManaAmount.scala

src/main/scala/com/github/unchama/seichiassist/subsystems/mana/domain/ManaManipulation.scala

...ain/scala/com/github/unchama/seichiassist/subsystems/mana/domain/LevelCappedManaAmount.scala

…sを変更

…mの修正

kuroma6666 · 2024-12-14T13:38:02Z

@rito528
ご指摘の内容を反映しました。
再度レビューよろしくお願いします。

rito528

Otherwise LGTM.

再レビュー申請前に sbt scalafixAll; scalafmtAll; を実行してもらえると助かります！

.../breaksuppressionpreference/application/BreakSuppressionPreferenceRepositoryDefinition.scala

...ms/breaksuppressionpreference/infrastructure/JdbcBreakSuppressionPreferencePersistence.scala

...ain/scala/com/github/unchama/seichiassist/subsystems/mana/domain/LevelCappedManaAmount.scala

...ain/scala/com/github/unchama/seichiassist/subsystems/breaksuppressionpreference/System.scala

src/main/scala/com/github/unchama/seichiassist/menus/skill/PassiveSkillMenu.scala

src/main/scala/com/github/unchama/seichiassist/listener/PlayerBlockBreakListener.scala

kuroma6666 · 2024-12-21T06:19:52Z

@rito528
レビューいただいた内容について修正・変更をかけたので、再度ご確認よろしくお願いします。

rito528

最後に1点だけ変更お願いします！

src/main/scala/com/github/unchama/seichiassist/menus/skill/PassiveSkillMenu.scala

Co-authored-by: rito528 <[email protected]>

rito528

LGTM.👍👍👍

お疲れ様でした！

kuroma6666 force-pushed the feat-digging-stop-mana-fully-consumed branch 2 times, most recently from 14a859f to 571abd3 Compare September 14, 2024 17:30

kuroma6666 marked this pull request as ready for review September 14, 2024 17:38

rito528 requested changes Sep 15, 2024

View reviewed changes

kuroma6666 force-pushed the feat-digging-stop-mana-fully-consumed branch 2 times, most recently from bb09299 to 509644b Compare September 15, 2024 06:27

kuroma6666 added 5 commits September 15, 2024 15:42

feat: パッシブスキルメニューにマナが無くなったら掘れなくするスキルのトグルボタンを追加

1fc70ab

feat: マナを消費しきった際の分岐処理を追加

09a17d6

chore: V1.19.0 ブロック破壊の抑制トリガー設定を保存するマイグレーションファイルを追加

924113e

feat: breakskilltriggerconfigサブシステムの追加

9f45f42

`breakskilltargetconfig`で扱えない事柄を扱う為のサブシステム

feat: ブロック破壊時、「マナ切れ」かどうか判定する処理を追加

e7c7cd3

kuroma6666 force-pushed the feat-digging-stop-mana-fully-consumed branch from 509644b to e7c7cd3 Compare September 15, 2024 06:46

rito528 requested changes Sep 15, 2024

View reviewed changes

src/main/scala/com/github/unchama/seichiassist/listener/PlayerBlockBreakListener.scala Outdated Show resolved Hide resolved

src/main/scala/com/github/unchama/seichiassist/util/BreakUtil.scala Outdated Show resolved Hide resolved

kuroma6666 added 3 commits September 16, 2024 12:56

fix: isBreakBlockManaFullyConsumedをbreakskilltriggerconfigサブシステムに実装

6725736

fix: スキル発動分のマナ不足のとき、ブロック破壊処理を終了するようにする

c043e8d

fix: 消費マナ不足処理の修正

f1e0599

kuroma6666 requested a review from rito528 September 21, 2024 07:48

rito528 requested changes Sep 21, 2024

View reviewed changes

kuroma6666 added 7 commits September 22, 2024 01:59

fix: PlayerBlockBreakListenerへマナ切れブロック破壊停止設定取得処理を移動

66d94e1

fix: マナ消費できるかどうか判断する関数を修正

acb3cf8

fix: isBreakBlockManaFullyConsumed関数をPlayerBlockBreakListenerへ定義

9753a7a

chore: importするパッケージの修正

05c80c1

fix: 消費マナ不足判定処理とブロック破壊時処理を明確にした

d79bad1

chore: scalafmt

ad5b883

chore: BreakSkillTriggerSettingsの削除

02c9368

kuroma6666 requested a review from rito528 September 22, 2024 13:19

kuroma6666 added 3 commits October 12, 2024 16:11

fix: 「マナがないときにスキルの発動を抑制するかどうか」のテーブル構造を修正

914bf47

fix isBreakBlockManaFullyConsumedの処理を修正

eaec16e

fix: 指摘内容(*)を反映し、onPlayerActiveSkillEventの処理を修正

df86061

*GiganticMinecraft#2370 (comment)

rito528 requested changes Oct 12, 2024

View reviewed changes

kuroma6666 added 7 commits December 14, 2024 15:35

fix: BreakSkillTriggerConfigKey->BreakSuppressionPreferenceへcase clas…

5d5bcd0

…sを変更

fix: BreakSkillTriggerConfigKey定義の削除

a4451d0

fix: BreakSkillTrigger->BreakSuppressionPreferenceへの変更に伴うsubsyste…

38c29da

…mの修正

fix: BreakSkillTrigger->BreakSuppressionPreferenceへの変更に伴うビルドエラー修正

7b3c3c7

fix: DBマイグレーションSQLの修正

a10401e

fix: tryConsumeの削除

1680ffe

fix: canAcuireの修正

95d772f

kuroma6666 requested a review from rito528 December 14, 2024 13:38

rito528 requested changes Dec 15, 2024

View reviewed changes

kuroma6666 added 7 commits December 21, 2024 14:24

fix: Repositoryに保存する場合のdoBreakSuppressionの初期値をドメイン定義に合わせた

f083c7d

fix: config名をoriginalBreakStopConfig->isBreakSuppressionEnabledに変更

8b3c52b

fix: toggleBreakSuppressionの記述を簡潔にした

95bf5f4

fix: doBreakSuppressionを削除しreadメソッドのロジックを簡素化

ffb920d

fix: isBreakBlockManaFullyConsumedをprivateに変更

c63fff9

chore: scalafmtAll,scalafixAll

41f616d

chore: 改行修正

a80e826

kuroma6666 requested a review from rito528 December 21, 2024 06:19

rito528 requested changes Dec 21, 2024

View reviewed changes

src/main/scala/com/github/unchama/seichiassist/menus/skill/PassiveSkillMenu.scala Outdated Show resolved Hide resolved

fix: isBreakSuppressionEnabledの条件判定を修正

1cfc829

Co-authored-by: rito528 <[email protected]>

kuroma6666 requested a review from rito528 December 21, 2024 06:39

rito528 approved these changes Dec 21, 2024

View reviewed changes

rito528 merged commit 3b84a4f into GiganticMinecraft:1_18 Dec 21, 2024
1 check passed

kuroma6666 deleted the feat-digging-stop-mana-fully-consumed branch December 22, 2024 08:48

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat: マナが無くなったら掘れなくするパッシブスキルを追加 #2370

feat: マナが無くなったら掘れなくするパッシブスキルを追加 #2370

kuroma6666 commented Aug 31, 2024 •

edited

Loading

kuroma6666 commented Sep 14, 2024

rito528 left a comment

kuroma6666 commented Sep 15, 2024

kuroma6666 commented Sep 15, 2024

rito528 left a comment

kuroma6666 commented Sep 21, 2024 •

edited

Loading

kuroma6666 commented Oct 12, 2024

kuroma6666 commented Dec 14, 2024

rito528 left a comment

kuroma6666 commented Dec 21, 2024

rito528 left a comment

rito528 left a comment

feat: マナが無くなったら掘れなくするパッシブスキルを追加 #2370

feat: マナが無くなったら掘れなくするパッシブスキルを追加 #2370

Conversation

kuroma6666 commented Aug 31, 2024 • edited Loading

このPRの変更点と理由:

主な変更点

補足情報:

kuroma6666 commented Sep 14, 2024

rito528 left a comment

Choose a reason for hiding this comment

kuroma6666 commented Sep 15, 2024

kuroma6666 commented Sep 15, 2024

rito528 left a comment

Choose a reason for hiding this comment

kuroma6666 commented Sep 21, 2024 • edited Loading

kuroma6666 commented Oct 12, 2024

kuroma6666 commented Dec 14, 2024

rito528 left a comment

Choose a reason for hiding this comment

kuroma6666 commented Dec 21, 2024

rito528 left a comment

Choose a reason for hiding this comment

rito528 left a comment

Choose a reason for hiding this comment

kuroma6666 commented Aug 31, 2024 •

edited

Loading

kuroma6666 commented Sep 21, 2024 •

edited

Loading