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

Implement #96 (app version changeable) #126

Closed

Conversation

davidlatwe
Copy link
Collaborator

This PR implements #96.

image

However, tech-debt increased, one more model interact directly with the controller object. I plan to solve that by merging PackagesModel and ApplicationModel in follow up PR, which I'd also like to make the version list be limited to profile requests range.

@davidlatwe
Copy link
Collaborator Author

Not mergeable.

Found a bug that when you select a version in main Apps view and click on other apps for triggering the comboBox data change, application request KeyError raised. Possible because application selection changed in the same time while context model reseting for version patching.

@davidlatwe
Copy link
Collaborator Author

Closing this one, just being superseded by #129.

@davidlatwe davidlatwe closed this Nov 11, 2020
@davidlatwe davidlatwe deleted the feature/#96-change-app-version branch November 13, 2020 06:51
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.

1 participant