-
Notifications
You must be signed in to change notification settings - Fork 25
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: update config parsing structure #319
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 🏎️
Please request a feedback from Volodymyr as well.
af8b8d5
to
5485df6
Compare
@volodymyr-chekyrta please spare some time to review this PR. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Any chance to merge it after the #308 changes? |
@volodymyr-chekyrta sure we can merge it after PR-308 and update the config structure accordingly. |
Thank you! |
- Update dictionary for ProgramConfig - Update UI related feature flags under a single Key - fix image load in course fix: LEARNER-9891
ea10ace
5485df6
to
ea10ace
Compare
dc13847
to
acd4a03
Compare
Description
fix: Issue-318