Skip to content

Update to V4#41

Open
Veyronity wants to merge 10 commits intomainfrom
v4
Open

Update to V4#41
Veyronity wants to merge 10 commits intomainfrom
v4

Conversation

@Veyronity
Copy link
Contributor

Based on #40

  • Move deprecated/old endpoints to /legacy/
  • Update endpoints to accept an API key, currently only the player & shop endpoints use it
  • Add QuickShop integratoin to allow players to lookup their own shops
  • Allow players to use their API keys to query their resident status, even if they have opted out
  • Add permissions for api command & key management

Also improve API version handling

Events:
- NewDay
- NationCreated, NationDeleted, NationKingChanged, NationMerged
- TownCreated, TownDeleted, TownRenamed, TownMayorChanged, TownMerged, TownRuined, TownReclaimed
- Update plugin version
- Update config.yml
- Move deprecated endpoints to /legacy/
- Update endpoints to accept an API key, currently only the player & shop endpoints use it
- Add QuickShop integratoin to allow players to lookup their own shops
- Allow players to use their API keys to query their resident status, even if they have opted out
- Add permissions for api command & key management
@Veyronity Veyronity requested a review from Warriorrrr March 6, 2026 13:26
Copy link

@Magg27 Magg27 left a comment

Choose a reason for hiding this comment

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

Minor grammar fixes in ApiCommand.java:

For lines:

  • 48: "You have opted back in for your information being public on the API."

> "You have opted back in to having your information public on the API."

  • 52: "You have opted out of your information being public on the API"

> "You have opted out of having your information public on the API."

  • 95: "You already have an API key! Use /api key to get it."

> "You already have an API key! Use /api key to view it."

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