-
Notifications
You must be signed in to change notification settings - Fork 2
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
springboard apps to be deployable #78
springboard apps to be deployable #78
Conversation
@Rizziepit review please |
@@ -0,0 +1,93 @@ | |||
[app:springboardfrontend] | |||
use = egg:springboard-{{ app_type }} |
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.
This should use _
instead of -
like https://github.com/universalcore/unicore-mc/pull/78/files#diff-5e7b9f95bfe899b8e57feed82a33ca49R38
pyramid.default_locale_name = {{default_language}} | ||
|
||
featured_languages = | ||
{{default_language}} |
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.
By default springboard will show the current language + featured languages. So when the default language is the current language, with this value, only the default language will be displayed in the header. Perhaps we need to put default_language
plus one or two other available languages here.
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.
@Rizziepit I was meant to remove this. We currently don't specify featured languages anymore in Mission Control so this should be blank. The fallback should be current language + 2 other
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.
Ok cool. Can you open an issue on springboard so that we implement the fallback?
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.
+1 when tests are passing again |
…d-apps-to-be-deployable springboard apps to be deployable
No description provided.