Skip to content

Prod Deployment Hotfixes#4

Merged
zaghadon merged 5 commits into
masterfrom
prod/deploy
Apr 11, 2026
Merged

Prod Deployment Hotfixes#4
zaghadon merged 5 commits into
masterfrom
prod/deploy

Conversation

@zaghadon
Copy link
Copy Markdown
Contributor

This pull request updates the deployment workflow and improves code quality and UI consistency across several components. The most significant changes include renaming and generalizing the deployment workflow for broader production use, updating secrets and variables for deployment, refining navigation highlighting logic, and making layout fixes for better responsiveness.

Deployment workflow improvements:

  • Renamed the deployment workflow file from .github/workflows/deploy-cpanel.yml to .github/workflows/deploy-prod.yml, updated its name to "Deploy To Production", and generalized concurrency group and environment variables to use FTP_SERVER and FTP_SERVER_DIR instead of cPanel-specific names. Also updated Node.js version to 24 and switched deployment secrets to the new naming convention. [1] [2] [3]
  • Updated the README.md to reflect the new deployment workflow filename, secrets, and variable names, ensuring documentation matches the code changes.

Navigation and UI logic:

  • Improved the navigation highlighting logic in Navbar.tsx by introducing a normalizePath helper and an isActiveLink function, ensuring correct active state for nested routes and removing trailing slashes inconsistencies. [1] [2] [3] [4]

Layout and responsiveness fixes:

  • Added min-w-0 utility classes and adjusted flex/grid layouts in StoriesCarousel.tsx to prevent overflow issues and improve responsiveness. [1] [2] [3] [4]
  • Made the offset card in ProcessSteps.tsx visible on all screen sizes (was previously hidden on non-xl screens).

Documentation formatting:

  • Fixed table formatting and minor wording in README.md for improved readability and consistency. [1] [2] [3] [4]

@zaghadon zaghadon merged commit ee69296 into master Apr 11, 2026
1 check passed
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.

1 participant