[Feature] Discord Integration w/ DiscordSRV #176
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
from #175
Please see the issue i create for explanations and my system overviews.
Important
I'm unsure how merging this would work.
So the Global Plot-System is pushing for a release which:
My suggesting is merging this pr and archive the main branch as 4.X.X, then force push the revamp in.
Leaving the pull request and let the release in ASEAN BTE fork also works.
Changes
The plugin is just an event system, it only register data provider from the plot-system database.
Communications is one-way, the discord plugin only receive events from plot-system and handles it case-by-case.
The plugin cannot modify plot-system data, anything managed by the plugin stays in its isolated database table.
New class:
com.alpsbte.plotsystem.utils.
DiscordUtilA singleton to manage all discord event system.
Example; Submitting a plot:
PlotUtils#abandonPlot
Plot-System/src/main/java/com/alpsbte/plotsystem/core/system/plot/utils/PlotUtils.java
Line 400 in 7e68767
Edited for each abandon plot to have scope info for the abandon reason.