Skip to content

System administration new design#58

Merged
tarekio merged 121 commits intomainfrom
BYNT-1333-Implement-Sys-Setting-new-design
Jul 10, 2025
Merged

System administration new design#58
tarekio merged 121 commits intomainfrom
BYNT-1333-Implement-Sys-Setting-new-design

Conversation

@apodacaduron
Copy link
Contributor

@apodacaduron apodacaduron commented Jan 14, 2025

Jira Issue

  1. 1333
  2. 1266

Description

image

Checklist

  • Tests added/updated
  • Documentation updated (if needed)
  • New strings prepared for translations

API Changes (if applicable)

  • Permissions checked
  • Endpoint tests added

Additional Notes

[Any other relevant information]

Copy link
Collaborator

@level09 level09 left a comment

Choose a reason for hiding this comment

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

This looks great. but can we fix this scenario in config state :

  1. go to system settings , save settings button is disabled
  2. go to maps tab, -> save settings button gets activated.

I think our geomap component needs a fix as it emits or changes the state of our config.

@apodacaduron
Copy link
Contributor Author

@level09 Thank you i've updated GeoMap to not emit if the initial coordinates are the exact same as the ones currently set in the map

@apodacaduron apodacaduron mentioned this pull request Jul 2, 2025
5 tasks
Copy link
Collaborator

@level09 level09 left a comment

Choose a reason for hiding this comment

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

System Administration

Small issue I Found, If i go to System Administration Dashboard, Maps tab and change only coordinates, the diff shows Diameter change to 1000 , maybe we need to fix default value bindings ? let me know if this is better done on the backend API.

Tab URLs and Router

It is nice to have the URL respond to the current active tab, but the history is not really functional, can we make it work ? like we do with drawers history ? (perhaps replace replace with push)

@apodacaduron apodacaduron requested a review from level09 July 2, 2025 23:30
Copy link
Contributor

@tarekio tarekio left a comment

Choose a reason for hiding this comment

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

Good overall, Import tools tab needs some work to match design.

@apodacaduron apodacaduron requested a review from tarekio July 10, 2025 17:38
@apodacaduron
Copy link
Contributor Author

@tarekio updated, import tools should better match figma now

Copy link
Contributor

@tarekio tarekio left a comment

Choose a reason for hiding this comment

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

Good to go.

@tarekio tarekio merged commit 9860148 into main Jul 10, 2025
6 of 7 checks passed
@tarekio tarekio deleted the BYNT-1333-Implement-Sys-Setting-new-design branch July 10, 2025 21:37
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.

3 participants