Skip to content

feat: add village overview and detail editing#1372

Open
Zhey-on wants to merge 1 commit intoEnAccess:mainfrom
Zhey-on:feat/1364-village-overview-detail
Open

feat: add village overview and detail editing#1372
Zhey-on wants to merge 1 commit intoEnAccess:mainfrom
Zhey-on:feat/1364-village-overview-detail

Conversation

@Zhey-on
Copy link
Copy Markdown

@Zhey-on Zhey-on commented Mar 12, 2026

Brief summary of the change made

This PR adds a proper Village management flow so villages are no longer create-only.

What is included:

  • A new Village Overview page at /villages to list existing villages.
  • A new Village Detail page at /villages/:id to view and update village data.
  • Route and sidebar integration for the new village pages.
  • City API integration on the frontend for detail and update operations.

closes: #1364

Are there any other side effects of this change that we should be aware of?

  • No backend endpoint changes were needed; this uses existing GET /api/cities/{id} and PUT /api/cities/{id} endpoints.
  • The village detail page is restricted to users with settings permission (consistent with village creation).

Describe how you tested your changes?

Manual verification in local development:

  1. Open /villages and confirm existing villages are listed.
  2. Click any village row and verify navigation to /villages/:id.
  3. Edit name/country/mini-grid as a settings user and save.
  4. Confirm success notification and persisted values after reload.
  5. Verify non-settings users can access view but cannot edit fields/save.

Pull Request checklist

Please confirm you have completed any of the necessary steps below.

  • Meaningful Pull Request title and description
  • Changes tested as described above
  • Added appropriate documentation for the change.
  • Created GitHub issues for any relevant followup/future enhancements if appropriate.

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.

[Feature Request]: Village Overview (and modification)

1 participant