Skip to content

removes unsure button (ORA-254) - #2084

Merged
adampalay merged 1 commit into
masterfrom
adam/remove_unsure_button
Jan 17, 2014
Merged

removes unsure button (ORA-254)#2084
adampalay merged 1 commit into
masterfrom
adam/remove_unsure_button

Conversation

@adampalay

Copy link
Copy Markdown
Contributor

removes the "I am unsure" button from peer evaluation and hardcodes it to false

@waheedahmed
@stephensanchez

@waheedahmed

Copy link
Copy Markdown
Contributor

👍 if we only have to remove it from front end.

@adampalay

Copy link
Copy Markdown
Contributor Author

@waheedahmed yeah just for now

@stephensanchez

Copy link
Copy Markdown
Contributor

👍 We should keep an open JIRA task to remove the underpinnings for ORA as well; with this change, I imagine it is all a dead code path, and removing it could simplify future debugging.

@waheedahmed

Copy link
Copy Markdown
Contributor

👍 Tested UI on fullstack and working fine.

@adampalay

Copy link
Copy Markdown
Contributor Author

Also tested in sandbox

adampalay added a commit that referenced this pull request Jan 17, 2014
@adampalay
adampalay merged commit 439037d into master Jan 17, 2014
@adampalay
adampalay deleted the adam/remove_unsure_button branch January 17, 2014 20:24
jenkins-ks pushed a commit to nttks/edx-platform that referenced this pull request Sep 7, 2017
* addition of test code of the certificate issuance function openedx#2038 (openedx#2054)

* Mod translation of 'Course End Date:' openedx#2084 (openedx#2090)

* Add menu to ga_operation for ga_analyzer openedx#2039 (openedx#2088)

* Fixed bugs openedx#2039 (openedx#2112)

* Fixed csv format openedx#2039 (openedx#2127)

* Change to split download if there are many display items openedx#916 (openedx#2121)

* Change to split download if there are many display items openedx#916

* Fix UT

* Fix Review

* Fix review2
shimulch pushed a commit to open-craft/openedx-platform that referenced this pull request Jan 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants