Chore/core 567 dashboard readme updates#296
Merged
Merged
Conversation
musayann
requested changes
Jul 17, 2026
|
|
||
| export const getFaqTutorialVideoUrl = () => getEnv(EnvVariables.FRONTEND_FAQ_TUTORIAL_VIDEO_URL); | ||
|
|
||
| export const getTevetSkillsHidden = () => getEnv(EnvVariables.FRONTEND_HIDE_TEVET_SKILLS).toLowerCase() === "true"; |
Contributor
There was a problem hiding this comment.
can we rename this to ProgramSkills instead. getProgramSkillsVisibility, FRONTEND_HIDE_PROGRAM_SKILLS
Contributor
There was a problem hiding this comment.
Pull request overview
This PR extends Compass Connect’s frontend configurability by adding (1) a country name used in user-facing dashboard copy via i18n interpolation, and (2) a feature flag to hide the TEVET programme-skills section across the dashboard/sidebar/profile. It also updates READMEs/assets and config injection so these values can be set from environment/config.
Changes:
- Add
GLOBAL_COUNTRY_NAMEand wire it into i18n default interpolation variables; update multiple locales to use{{country}}. - Add
FRONTEND_HIDE_TEVET_SKILLSfeature flag and apply it to the home sidebar, skills-discovery sidebar, and profile “Education Skills” section. - Update config/injection/docs/examples (and top-level README logos) to reflect the new branding/config options.
Reviewed changes
Copilot reviewed 23 out of 23 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| README.md | Switches README logo assets to repo-hosted SVGs and adjusts display size. |
| iac/templates/env.template | Documents new env vars (FRONTEND_HIDE_TEVET_SKILLS, GLOBAL_COUNTRY_NAME). |
| iac/frontend/prepare_frontend.py | Emits the new env vars into the generated frontend env JS payload. |
| frontend-new/src/profile/components/SkillsDiscoveredCard/SkillsDiscoveredCard.tsx | Hides the “Education Skills” (TEVET) section when the flag is enabled and adjusts indices accordingly. |
| frontend-new/src/home/components/Sidebar/SkillsDiscoverySidebar.tsx | Hides programme skills block (and related “any skills” logic) behind the new flag. |
| frontend-new/src/home/components/Sidebar/HomeSidebar.tsx | Hides programme skills section behind the new flag. |
| frontend-new/src/envService.ts | Adds GLOBAL_COUNTRY_NAME + FRONTEND_HIDE_TEVET_SKILLS getters and env enum entries. |
| frontend-new/src/i18n/i18n.ts | Adds country as a default i18n interpolation variable via getCountryName(). |
| frontend-new/src/i18n/locales/en-US/translation.json | Replaces hardcoded country mentions with {{country}}. |
| frontend-new/src/i18n/locales/en-GB/translation.json | Replaces hardcoded country mentions with {{country}}. |
| frontend-new/src/i18n/locales/es-ES/translation.json | Replaces hardcoded country mentions with {{country}}. |
| frontend-new/src/i18n/locales/es-AR/translation.json | Replaces hardcoded country mentions with {{country}}. |
| frontend-new/src/i18n/locales/pt-MZ/translation.json | Replaces hardcoded country mentions with {{country}}. |
| frontend-new/src/i18n/locales/sw-KE/translation.json | Replaces hardcoded country mentions with {{country}}. |
| frontend-new/src/i18n/locales/ny-ZM/translation.json | Replaces hardcoded country mentions with {{country}}. |
| frontend-new/src/_test_utilities/envServiceMock.ts | Extends envService mocks with getCountryName and getTevetSkillsHidden. |
| frontend-new/src/home/components/Footer/snapshots/Footer.test.tsx.snap | Updates snapshot output to reflect the new {{country}} token in translations. |
| frontend-new/README.md | Documents FRONTEND_HIDE_TEVET_SKILLS. |
| frontend-new/public/data/env.example.js | Adds example values for the new env vars. |
| config/inject-config.py | Maps GLOBAL_COUNTRY_NAME to branding.country for injection. |
| config/default.json | Adds branding.country. |
| config/njira.json | Adds branding.country. |
| config/CUSTOMIZATION.md | Documents branding.country customization. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Fidesnoella
force-pushed
the
chore/CORE-567-dashboard-readme-updates
branch
from
July 17, 2026 14:30
73ccab9 to
272b078
Compare
musayann
approved these changes
Jul 17, 2026
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.
Ticket: CORE-567