Feat/navbar external services - #213
Open
martimgil wants to merge 3 commits into
Open
Conversation
…ervices
Rally moved off the platform into a standalone project
(Platform-rally-extension) and Gamification (nei-gamification-system) was
never wired into the navbar. Both are now shown as plain-text nav items,
env-gated, with a lightweight health probe that disables the link (opacity,
non-clickable) when the service is unreachable instead of hiding it -
avoids false negatives from network/CORS blips.
- web-nei: new useServiceHealth hook, wired into Navbar for Rally (single
target) and Gamification (web + api-game/api-shop/api-chat, since it's a
microservice architecture)
- web-nei/config: WEB_RALLY_URL now comes from VITE_RALLY_URL instead of
the dead in-cluster ${BASE_URL}/rally path; new VITE_GAMIFICATION_URL/
VITE_ENABLE_GAMIFICATION
- Remove dead Rally extension remnants left over from the extraction:
untracked extensions/rally/ clone, orphaned .git/modules/extensions/rally
worktree data, stale Rally sections in EXTENSIONS.md and
dev/proxy/README.md, stale docstring example in extension_scopes.py, dead
specialData block and commented route in web-nei
- .gitignore: also ignore leftover rally worktree-session clones
…ate Gamification label
|
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.
This pull request updates the NEI Platform to reflect that the Rally Tascas and Gamification features are no longer embedded platform extensions, but instead are managed as independent, external services. The documentation, configuration, and frontend have all been updated to remove Rally as an embedded extension and to support linking to these services externally via environment variables. Additionally, a new frontend hook has been added to check the health of these external services and display their status in the navigation bar.
Key changes include:
Documentation and Extension Management Updates:
EXTENSIONS.mdand related documentation to remove Rally as an embedded extension, clarify that it is now an external service, and update all examples and instructions to reference only the Gala extension as an embeddable extension. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10] [11] [12]Configuration and Environment Variables:
compose.ymlandweb-nei/.envto add environment variables for enabling/disabling Rally and Gamification as external services, and for specifying their public URLs. [1] [2]web-nei/src/config/index.jsx) to read these environment variables and expose them to the app.Frontend Navigation and Health Checking:
useServiceHealth, to probe and display the online status of external services in the navigation bar. [1] [2] [3]Code Cleanup:
These changes ensure that Rally and Gamification are now treated as independent services, decoupled from the platform extension system, and can be linked and monitored via configuration.