Skip to content
This repository has been archived by the owner on Jul 23, 2019. It is now read-only.

fix(community-badge): community badge should appear for missions #284

Merged
merged 2 commits into from
Jun 18, 2018

Conversation

ia3andy
Copy link
Contributor

@ia3andy ia3andy commented Jun 18, 2018

No description provided.

@ghost ghost assigned ia3andy Jun 18, 2018
@ghost ghost added the in progress label Jun 18, 2018
@ia3andy ia3andy requested review from invincibleJai and edewit June 18, 2018 13:32
@@ -190,6 +190,7 @@ export class MissionRuntimeCreateappStepComponent extends LauncherStep implement
if (this.missionId === mission.id && availableBoosters.empty) {
this.clearMission();
}
mission.community = this.launcherComponent.flow === 'osio' && this.versionId === 'community';
Copy link
Collaborator

@invincibleJai invincibleJai Jun 18, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ia3andy can we have the check for disabled as well i.e mission.disabled to be false

@edewit edewit merged commit 9b4b24c into fabric8-launcher:master Jun 18, 2018
@ghost ghost removed the in progress label Jun 18, 2018
Copy link
Member

@edewit edewit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ia3andy
Copy link
Contributor Author

ia3andy commented Jul 9, 2018

🎉 This PR is included in version 1.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

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.

3 participants