New spells Spiritus and Expelliarmus#12
New spells Spiritus and Expelliarmus#12HyScript7 merged 14 commits intoHyScript7:feat/content_update_1from
Conversation
|
Can you change the target branch to the content update one? Thanks. |
|
turns out I can do it myself |
Dw I am adding 2 more things rn so wait |
Lmao |
|
Alright I am done read docs if u need more help, rest has already been reviewed by copilot also i checked twice on localhost all works, u can read docs if everything is good, then u can merge right now ( merge little soon btw ) |
src/main/java/io/github/youngcoder45/ascendancy/spells/Ignis.java
Outdated
Show resolved
Hide resolved
src/main/java/io/github/youngcoder45/ascendancy/spells/Ignis.java
Outdated
Show resolved
Hide resolved
| @EventHandler(priority = EventPriority.LOWEST) | ||
| public void onDebugChat(AsyncChatEvent event) { | ||
| String message = PlainTextComponentSerializer.plainText().serialize(event.message()); | ||
|
|
There was a problem hiding this comment.
Please add an operator check. Then we can merge.
There was a problem hiding this comment.
Also since you wrote this in my package, either move it or add a copyright comment at the start of the file.
|
Are you adding op perms listeners or should I just delete that file myself rn? |
|
Bro atleast reply @HyScript7 |
|
Probably delete it. We'll add it sometime else when we actually need debug mode. |
|
Merged manually in 5745a8e |
2 participants
No description provided.