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

Don't show suggested/recommended for mods that can't be installed #1427

Merged
merged 2 commits into from
Aug 16, 2016
Merged

Don't show suggested/recommended for mods that can't be installed #1427

merged 2 commits into from
Aug 16, 2016

Conversation

Postremus
Copy link
Contributor

We don't show an empty suggested and recommended list if we can't install any of the mods.
Also don't show the recommended / suggested page if the user already cancelled the Installation.

@pjf pjf changed the title Recommended page fixes Don't show suggested/recommended if those mods can't be installed Sep 27, 2015
@Olympic1 Olympic1 added the GUI Issues affecting the interactive GUI label May 18, 2016
@Postremus Postremus changed the title Don't show suggested/recommended if those mods can't be installed Don't show suggested/recommended for mods that can't be installed Jun 29, 2016
@politas
Copy link
Member

politas commented Jul 13, 2016

I'm trying to merge this, and it seems to lose the buildID checking (and possibly other things). Can you re-base this? (I tried rebasing it myself and got some build errors)

@Postremus
Copy link
Contributor Author

done

@politas politas merged commit 8669881 into KSP-CKAN:master Aug 16, 2016
politas added a commit that referenced this pull request Aug 16, 2016
@politas
Copy link
Member

politas commented Aug 16, 2016

Merged with thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
GUI Issues affecting the interactive GUI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants