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

Remove hardcoded config data from webui #1323

Merged
merged 4 commits into from
Sep 6, 2019

Conversation

kschiffer
Copy link
Contributor

Summary

Closes #1154

Changes

  • Replace hardcoded references to The Things Stack Console with environment data
    • both in oauth and console
  • Minor related changes

Notes for Reviewers

The logo still uses The Things Stack Logo as alt value. I think it does not make sense to replace that as the logo in itself is "hardcoded" as well, and will stay like that for now.

@kschiffer kschiffer added c/console This is related to the Console c/identity server This is related to the Identity Server prio/medium ui/web This is related to a web interface labels Sep 5, 2019
@kschiffer kschiffer added this to the September 2019 milestone Sep 5, 2019
@kschiffer kschiffer self-assigned this Sep 5, 2019
Copy link
Contributor

@bafonins bafonins left a comment

Choose a reason for hiding this comment

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

Screenshot 2019-09-05 at 18 00 48

@kschiffer kschiffer force-pushed the feature/1154-remove-hardcoded-data branch from 3a8a0b2 to 5ba8058 Compare September 5, 2019 15:34
@kschiffer kschiffer force-pushed the feature/1154-remove-hardcoded-data branch from 5ba8058 to 419c824 Compare September 5, 2019 15:49
@kschiffer kschiffer merged commit 559e157 into master Sep 6, 2019
@kschiffer kschiffer deleted the feature/1154-remove-hardcoded-data branch September 6, 2019 08:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c/console This is related to the Console c/identity server This is related to the Identity Server ui/web This is related to a web interface
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Remove hardcoded references to configurable data
2 participants