Conversation
* login, dashboard and profile done * refactored * auth package comes from public npm * changed frontend formatting * fixed an npm bug * Update package.json Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * template fe and translate packages come from public npm * ran npm i * updated files to use public packages * fixed imports * fix: fixed dependencies calls across the entire source code -frontend-, and updated config files * ops: created dockerignore for root context and updated worfklow cd for backend stack --------- Co-authored-by: a-elkhiraooui-ciscode <a.elkhiraoui@ciscod.com> Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
* login, dashboard and profile done * refactored * auth package comes from public npm * changed frontend formatting * fixed an npm bug * Update package.json Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * template fe and translate packages come from public npm * ran npm i * updated files to use public packages * fixed imports * fix: fixed dependencies calls across the entire source code -frontend-, and updated config files * ops: created dockerignore for root context and updated worfklow cd for backend stack * ops: update CI workflows names for status checks * ops: updated ci workflows names --------- Co-authored-by: a-elkhiraooui-ciscode <a.elkhiraoui@ciscod.com> Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Contributor
There was a problem hiding this comment.
Pull request overview
This PR integrates authentication, internationalization, and UI components into the frontend application, transitioning from a basic placeholder to a functional multi-language application with routing and layout infrastructure.
Changes:
- Added authentication provider with OAuth support (Google, Microsoft) and session management
- Implemented i18n support for English, French, and Arabic with translation resources
- Integrated UI component libraries (@ciscode/template-fe, @ciscode/ui-authentication-kit) with custom Tailwind configuration
- Set up routing infrastructure with loading splash screen and protected app layout
- Updated build configurations and CI/CD workflows with environment variable handling
Reviewed changes
Copilot reviewed 40 out of 48 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| package.json | Added concurrently for parallel dev server execution and new workspace scripts |
| frontend/vite.config.ts | Added publicDir configuration for static assets |
| frontend/tailwind.config.cjs | Replaced simple config with comprehensive theme including custom colors, fonts, and UI kit paths |
| frontend/src/pages/Loading.tsx | New splash screen component with session storage tracking and auth-aware navigation |
| frontend/src/main.tsx | Integrated AuthProvider, I18nProvider, and routing providers with brand configuration |
| frontend/src/i18n/*.json | Added translation resources for auth, menu, and template libraries in en/fr/ar |
| frontend/src/i18n/loadAll.ts | Consolidated i18n resource imports using JSON assertions |
| frontend/src/app/* | New router and layout components implementing authenticated application structure |
| frontend/package.json | Downgraded @vitejs/plugin-react and replaced rolldown-vite with standard vite |
| .github/workflows/*.yml | Simplified workflow names and added environment variable handling for deployment |
| backend/docker/Dockerfile | Build context changed from backend/ to monorepo root |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Ciscode-Admin
approved these changes
Jan 28, 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.
No description provided.