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

fix: allow to set environment in Config.instance #98

Conversation

shiipou
Copy link
Contributor

@shiipou shiipou commented Sep 25, 2023

About this PR

Closes https://github.com/lenra-io/kubernetes/issues/130

We can't set environment at runtime for client and backoffice.

This PR allow to set in the index.html file the metadata environment with a value at runtime instead of being stuck at compile time.

How to test my changes

Checklist

  • I didn't over-scope my PR
  • My PR title matches the commit convention
  • I did not include breaking changes
  • I made my own code-review before requesting one

I included unit tests that cover my changes

  • 👍 yes
  • 🙅 no, because they aren't needed
  • 🙋 no, because I need help

I added/updated the documentation about my changes

  • 📜 README.md
  • 📕 docs/*.md
  • 📓 docs.lenra.io
  • 🙅 no documentation needed

@shiipou shiipou self-assigned this Sep 25, 2023
lib/config/config.dart Outdated Show resolved Hide resolved
@taorepoara taorepoara merged commit 76b91ec into beta Sep 29, 2023
2 checks passed
@taorepoara taorepoara deleted the kubernetes-130-bug-sentry-client-backoffice-does-not-have-an-environment branch September 29, 2023 06:10
taorepoara pushed a commit that referenced this pull request Sep 29, 2023
* fix: allow to set environment in Config.instance

Signed-off-by: shiipou <[email protected]>

* Update lib/config/config.dart

Co-authored-by: Jonas Martinez <[email protected]>

---------

Signed-off-by: shiipou <[email protected]>
Co-authored-by: Jonas Martinez <[email protected]>
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