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

E1786. Bookmark Code Fix #1039

Closed
wants to merge 9 commits into from
Closed

E1786. Bookmark Code Fix #1039

wants to merge 9 commits into from

Conversation

aapandey
Copy link

->In this code fix, we have resolved the issue of back button not functioning properly when a student navigates through the bookmarks and clicks on the back button.
->Code fix is also about adding questionnaire and fixing the rubric system that has trouble with bookmark selection and creation of new questionnaires.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 50.851% when pulling 8c952a5 on aapandey:master into a26cf3a on expertiza:master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.02%) to 50.829% when pulling 2199bbf on aapandey:master into a26cf3a on expertiza:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 50.851% when pulling 3727a5b on aapandey:master into a26cf3a on expertiza:master.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.01%) to 50.862% when pulling 4b90602 on aapandey:master into a26cf3a on expertiza:master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.6%) to 50.273% when pulling 4f63e52 on aapandey:master into a26cf3a on expertiza:master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.6%) to 50.273% when pulling 2ac016b on aapandey:master into a26cf3a on expertiza:master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.6%) to 50.273% when pulling 2ac016b on aapandey:master into a26cf3a on expertiza:master.

@Winbobob Winbobob force-pushed the master branch 3 times, most recently from 2cc51d9 to a1b78e0 Compare November 15, 2017 21:01
@@ -70,9 +70,7 @@

<br />

<%= link_to 'New bookmark', :action => 'new', :id => @topic.id %> |
<%= link_to 'Back', :controller=> 'sign_up_sheet', :action=>'list', :assignment_id => @topic.assignment_id %>
Copy link
Member

Choose a reason for hiding this comment

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

Could you explain why you remove the back link?

@Winbobob Winbobob closed this Jan 8, 2018
@Winbobob Winbobob changed the title Bookmark Code Fix E1786. Bookmark Code Fix Jan 31, 2018
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.

None yet

4 participants