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

Fix #1697: upgrade collection schema version to 2 by introducing language codes to collections (where the default is 'en'). #1856

Merged
merged 2 commits into from
May 19, 2016

Conversation

BenHenning
Copy link
Member

Introduced a language_code to the collection data structure and updated its schema version to 2, where new collection have a default language code of 'en'. The support for this language code has been implemented end-to-end for collections, including in the collection creation dialog and for search queries (though this commit does not implement collection searching).

its schema version to 2, where new collection have a default language
code of 'en'. The support for this language code has been implemented
end-to-end for collections, including in the collection creation dialog
and for search queries (though this commit does not implement collection
searching).
@BenHenning
Copy link
Member Author

/cc @amgowano

@@ -198,7 +201,7 @@ oppia.factory('CollectionUpdateService', [
},

/**
* Changes the title of a category and records the change in the
Copy link
Member

Choose a reason for hiding this comment

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

lol :)

@seanlip
Copy link
Member

seanlip commented May 19, 2016

LGTM -- looks fantastic. Thanks @BenHenning! OK to merge when Travis checks pass.

@BenHenning
Copy link
Member Author

Thanks for the review, @seanlip!

@BenHenning BenHenning merged commit ae6db1e into develop May 19, 2016
@BenHenning BenHenning deleted the fix-1697 branch May 19, 2016 18:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants