Add MiniGrid overview page and update map routes.#1192
Open
Kiwumulo-Nickson wants to merge 4 commits intoEnAccess:mainfrom
Open
Add MiniGrid overview page and update map routes.#1192Kiwumulo-Nickson wants to merge 4 commits intoEnAccess:mainfrom
Kiwumulo-Nickson wants to merge 4 commits intoEnAccess:mainfrom
Conversation
dmohns
reviewed
Dec 10, 2025
Member
There was a problem hiding this comment.
thanks a lot for your contribution 🙏 . Dropping a couple of comments here!
- You can use Github Pull Requests keywords to automatically link this PR to an issue: https://docs.github.com/en/get-started/writing-on-github/working-with-advanced-formatting/using-keywords-in-issues-and-pull-requests For example
Resolves: #130in the description of the PR.
dmohns
requested changes
Jan 6, 2026
Member
dmohns
left a comment
There was a problem hiding this comment.
Hey @Kiwumulo-Nickson just doing some house-keeping on open PRs. Please see comment above.
Member
|
Hey @Kiwumulo-Nickson You still have some merge conflicts here. Also, more leftovers from the Africa's Talking docs. Could you clean up both of these? |
dmohns
reviewed
Mar 30, 2026
| }, | ||
| }, | ||
| name: "MiniGridDashboardPage", | ||
| components: { MiniGridDashboard }, |
Member
There was a problem hiding this comment.
I think there is some Syntax error here. MiniGridDashboard is not imported anywhere.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Brief summary of the change made
Resolves: #130
#Created MiniGridDashboard.vue component — similar to AgentDashboard.vue:
Metrics boxes: Total Mini-Grids, Total Clusters, Total Cities, Total Agents
Searchable table listing all mini-grids
Columns: Name, Cluster, Created At, Cities, Agents
Clickable rows navigate to the detail page
#Refresh button to reload data
#Updated pages/Dashboard/MiniGrid/index.vue:
#Replaced the redirect logic with the new MiniGridDashboard component which matches the Agents overview page structure.
#Added translation keys to en.json:
miniGridDashboard: "Mini-Grid Overview"
miniGridList: "Mini-Grid List"
totalMiniGrids: "Total Mini-Grids"
totalClusters: "Total Clusters"
totalCities: "Total Cities"
#The component handles missing API data (e.g., cities_count, agents_count, cluster) and displays "-" or 0 when unavailable. The overview page now shows a list of mini-grids in a table format, matching the Agents overview page pattern, instead of using a dropdown menu.
Are there any other side effects of this change that we should be aware of?
Describe how you tested your changes?
-I used es-linters to test it out for any errors , and first run the setup locally on my machine before pushing the code to the remote repo .
Pull Request checklist
Please confirm you have completed any of the necessary steps below.