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

Simplify config options #151

Closed
2 tasks done
acelaya opened this issue Jan 29, 2022 · 0 comments · Fixed by #165
Closed
2 tasks done

Simplify config options #151

acelaya opened this issue Jan 29, 2022 · 0 comments · Fixed by #165
Labels
enhancement New feature or request tech-debt
Milestone

Comments

@acelaya
Copy link
Member

acelaya commented Jan 29, 2022

  • Remove the deprecated config approach, and make them support env vars only.
  • Simplify how to check which config is already set. Now the generated config is a plain map, so there's no need to use PathCollections
@acelaya acelaya added enhancement New feature or request tech-debt labels Jan 29, 2022
@acelaya acelaya added this to the 7.2.0 milestone Jan 29, 2022
@acelaya acelaya added this to Shlink Aug 1, 2022
@acelaya acelaya moved this to In review in Shlink Aug 1, 2022
@acelaya acelaya moved this from In review to Done in Shlink Aug 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request tech-debt
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

1 participant