Skip to content

🔧(backend) configurable footer #841

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Apr 4, 2025
Merged

Conversation

AntoLC
Copy link
Collaborator

@AntoLC AntoLC commented Apr 4, 2025

Purpose

  • We want to have different footer content per environment.
  • We want to be able to not have footer as well

Proposal

  • We added FRONTEND_FOOTER_FEATURE_ENABLED environment variable, a feature flag to enable or not the footer.
  • We added the FRONTEND_URL_JSON_FOOTER environment variable. It will give the possibility to generate your own footer content in the frontend.

Structure content footer:

https://github.com/suitenumerique/docs/blob/1972ef4d07eb2be2b7b0ac901b00ac8ffc8b9d7f/src/frontend/apps/impress/public/contents/footer-demo.json

@AntoLC AntoLC added the backend label Apr 4, 2025
@AntoLC AntoLC marked this pull request as draft April 4, 2025 08:05
@lunika lunika force-pushed the backend/footer-configurable branch 2 times, most recently from 9c19d91 to d6fa2b8 Compare April 4, 2025 09:21
AntoLC and others added 2 commits April 4, 2025 15:08
We added the `FRONTEND_URL_JSON_FOOTER` environment
variable. It will give the possibility to generate
your own footer content in the frontend.
If the variable is not set, the footer will not
be displayed.
We move the cache reset in the global conf test to not have to think
about reseting the cache when we implement test.
@AntoLC AntoLC force-pushed the backend/footer-configurable branch from 1972ef4 to 087eca3 Compare April 4, 2025 13:08
@AntoLC AntoLC changed the title 🔧(backend) add FRONTEND_URL_JSON_FOOTER env var 🔧(backend) configurable footer Apr 4, 2025
@AntoLC AntoLC marked this pull request as ready for review April 4, 2025 13:09
@AntoLC AntoLC force-pushed the backend/footer-configurable branch from 087eca3 to 9054cf8 Compare April 4, 2025 13:10
@AntoLC AntoLC self-assigned this Apr 4, 2025
@AntoLC AntoLC requested a review from lunika April 4, 2025 13:16
@@ -63,5 +63,6 @@ COLLABORATION_SERVER_SECRET=my-secret
COLLABORATION_WS_URL=ws://localhost:4444/collaboration/ws/

# Frontend
FRONTEND_FOOTER_FEATURE_ENABLED=True
Copy link
Member

Choose a reason for hiding this comment

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

Add it line 67 to group with FRONTEND_URL_JSON_FOOTER ?

We added the feature flag `FRONTEND_FOOTER_FEATURE_ENABLED`
to enable or disable the footer in the frontend.
@AntoLC AntoLC force-pushed the backend/footer-configurable branch from 9054cf8 to f03784b Compare April 4, 2025 13:32
@AntoLC AntoLC enabled auto-merge (rebase) April 4, 2025 13:32
@AntoLC AntoLC merged commit c11d59c into main Apr 4, 2025
19 of 20 checks passed
@AntoLC AntoLC deleted the backend/footer-configurable branch April 4, 2025 13:44
@AntoLC AntoLC mentioned this pull request Apr 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants