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 #5798: Introduce Improvements Tab to Exploration Editor #5991

Merged
merged 142 commits into from
Jan 1, 2019

Conversation

brianrodri
Copy link
Contributor

@brianrodri brianrodri commented Dec 16, 2018

Improvements Tab Design Doc

This PR introduces the boilerplate to show the Improvements Tab on the exploration editor behind an admin-config boolean. When on, an additional "tab" will appear in the exploration editor's navigation.

The Improvements Tab is intentionally hard-coded; it will be populated in the future with meaningful data. The config value will be removed once the Improvements Tab is considered "complete".

Screenshots:
Improvements Tab
Admin config for Improvements Tab

@brianrodri brianrodri changed the title Fix #5798: Introduce Issues Tab to Exploration Editor Fix #5798: Introduce Improvements Tab to Exploration Editor Dec 19, 2018
Copy link
Contributor

@nithusha21 nithusha21 left a comment

Choose a reason for hiding this comment

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

Looks good to me! Will await @BenHenning's review :)

@brianrodri brianrodri dismissed BenHenning’s stale review December 19, 2018 08:56

All change requests have been resolved; awaiting next review.

@brianrodri brianrodri requested review from aks681 and BenHenning and removed request for BenHenning December 31, 2018 08:23
@brianrodri brianrodri merged commit faf68df into oppia:develop Jan 1, 2019
@brianrodri brianrodri deleted the introduce-issues-tab branch January 1, 2019 20:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants