Skip to content
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

extract environment banner functionality and component to seperate repo #318

Merged
merged 7 commits into from
Sep 6, 2022

Conversation

elpoelma
Copy link
Contributor

@elpoelma elpoelma commented Aug 29, 2022

This PR replaces the internal implementation of the environment banner by the extracted one found in https://github.com/lblod/ember-environment-banner. This ensures that the environment banner can be easily reused across different lblod projects. The extracted banner also already uses the changes from #314.
The environmentName variable should be explicitly set for in order for the banner to be displayed.

@elpoelma elpoelma requested review from Dietr, nvdk and abeforgit August 29, 2022 13:06
@elpoelma elpoelma marked this pull request as ready for review September 2, 2022 09:05
Base automatically changed from fix/environment-banner to master September 6, 2022 17:49
@abeforgit abeforgit merged commit 3460d90 into master Sep 6, 2022
@abeforgit abeforgit deleted the internal/environment-banner-extraction branch September 6, 2022 18:20
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.

3 participants