Skip to content

Develop#36

Merged
Ciscode-Admin merged 16 commits intomasterfrom
develop
Jan 29, 2026
Merged

Develop#36
Ciscode-Admin merged 16 commits intomasterfrom
develop

Conversation

@Zaiidmo
Copy link
Contributor

@Zaiidmo Zaiidmo commented Jan 29, 2026

No description provided.

Zaiidmo and others added 15 commits January 21, 2026 11:44
* chore: install CiscodeAuthPackage, Mongoose, and env Depps

* Auth: Inject AuthKit into main app module, and init the db seeder for default roles and permissions

* docs: update Readme

* fix: build(backend)- use root workspace lockfile in Docker build

* fix: docker image build
* 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>
@Zaiidmo Zaiidmo requested review from a team and Copilot January 29, 2026 08:40
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR consolidates development tooling and branding updates across the monorepo. It introduces unified build/dev scripts at the root level, updates branding assets to use local files instead of external CDN URLs, adds environment configuration for MongoDB, and streamlines CI/CD workflows with improved secret management.

Changes:

  • Added concurrent development scripts and build commands for frontend/backend workspaces
  • Replaced external CDN logo references with local assets (/logo.png, /favicon.png)
  • Enhanced backend deployment to sync multiple secrets to SSM Parameter Store and inject them as environment variables

Reviewed changes

Copilot reviewed 11 out of 17 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
package.json Added workspace-level dev/build scripts and concurrently dependency
frontend/vite.config.ts Configured public directory for static assets
frontend/src/pages/Loading.tsx Updated logo source to local file
frontend/src/main.tsx Changed logoUrl to local favicon
frontend/index.html Updated page title and favicon reference
backend/src/core/config/env.schema.ts Added optional MONGO_URI environment variable
.github/workflows/frontend-ci.yml Simplified workflow name
.github/workflows/frontend-cd.yml Added frontend environment variable injection during build
.github/workflows/cdk-synth.yml Simplified workflow name
.github/workflows/backend-ci.yml Simplified workflow name
.github/workflows/backend-cd.yml Replaced single MongoDB secret with multi-secret SSM sync and dynamic environment injection

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Ciscode-Admin
Ciscode-Admin previously approved these changes Jan 29, 2026
@Ciscode-Admin Ciscode-Admin merged commit a212dff into master Jan 29, 2026
6 checks passed
@Ciscode-Admin Ciscode-Admin deleted the develop branch January 29, 2026 08:44
@Zaiidmo Zaiidmo restored the develop branch January 29, 2026 11:43
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