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

Collections should have a language code #1697

Closed
seanlip opened this issue Apr 7, 2016 · 7 comments
Closed

Collections should have a language code #1697

seanlip opened this issue Apr 7, 2016 · 7 comments
Assignees
Milestone

Comments

@seanlip
Copy link
Member

seanlip commented Apr 7, 2016

Currently, explorations have language codes specified (in addition to title, objective, etc.) whereas collections don't. However, in order to determine which collections to show to users requesting learning material in a particular language, we'll need to know the language codes of the collections.

The aim of this issue is therefore to add a language code property to the collection storage model. It should mimic that for explorations. In addition, it should be possible for authors of collections to set the language code explicitly in the collection editor's settings tab.

We might also want to add a warning within the collection editor if a constituent exploration has a language code that's different from the collection's language code.

/cc @BenHenning

@BenHenning
Copy link
Member

Adding an additional note for context: currently, the collection editor does not have a concept of warnings. It issues either warnings through the alerts service (which are essentially hard failures) or enumerates validation errors, which must be fixed before saving the collection. The warning for the language code should be a soft warning, meaning the editor should not prevent the collection from being saved if the warning is present, even if the collection is published.

Also, the collection editor does not currently have a functional settings tab, but it will soon. :)

@seanlip
Copy link
Member Author

seanlip commented May 2, 2016

Hi @hitesh96db, would you be interested in looking into this?

@hitesh96db
Copy link
Contributor

Sure :)

@seanlip
Copy link
Member Author

seanlip commented May 2, 2016

Thanks! Assigned you :-)

@seanlip
Copy link
Member Author

seanlip commented May 2, 2016

Btw this comment explains how to get started with collections: #1587 (comment)

@BenHenning
Copy link
Member

Note: my previous comment is no longer true. We now have a CollectionValidatorService for doing the check Sean mentioned.

@BenHenning
Copy link
Member

Since this is now blocking #1630, I'm going to take it over.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants