Skip to content
This repository has been archived by the owner on Dec 12, 2023. It is now read-only.

Fix update dynamic flags #100

Merged
merged 2 commits into from
Oct 13, 2023
Merged

Fix update dynamic flags #100

merged 2 commits into from
Oct 13, 2023

Conversation

adriantpaez
Copy link
Contributor

Changes:

  • Fix dynamic options flags in the update procees. The default value of the flag should be the current option value if it has one. If not, the default option value should be used.

Types of changes

  • Bugfix (non-breaking change which fixes an issue)

Testing

Requires testing Yes/No

In case you checked yes, did you write tests? No, the current tests are updated.

@adriantpaez adriantpaez requested a review from AntiD2ta as a code owner October 11, 2023 12:26
@adriantpaez adriantpaez self-assigned this Oct 11, 2023
@AntiD2ta AntiD2ta force-pushed the fix/update-dynamic-flags branch from 35a6e55 to aa2f340 Compare October 13, 2023 06:15
@AntiD2ta AntiD2ta merged commit d62087c into develop Oct 13, 2023
@AntiD2ta AntiD2ta deleted the fix/update-dynamic-flags branch October 13, 2023 15:02
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants