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

Course lesson title edit not working in content area #109

Open
brylie opened this issue Sep 8, 2015 · 8 comments
Open

Course lesson title edit not working in content area #109

brylie opened this issue Sep 8, 2015 · 8 comments

Comments

@brylie
Copy link
Member

brylie commented Sep 8, 2015

Observed behavior

When editing a course lesson title, via the inline editor in the content area, the changes are not saved.

Expected behavior

Updates to the course title, made via the content area inline editor, should save to the database.

@sebbel
Copy link
Contributor

sebbel commented Sep 8, 2015

Which branch are we talking about here? The develop branch?

@brylie
Copy link
Member Author

brylie commented Sep 8, 2015

Yep. Basically, editing the lesson title via the sidebar works, but not the main lesson section.

@brylie
Copy link
Member Author

brylie commented Sep 8, 2015

It turns out to be a small fix, so I will have a PR shortly.

@sebbel
Copy link
Contributor

sebbel commented Sep 8, 2015

When I press the "Edit Course" button, nothing happens.
tested on branch develop
Chrome
Firefox

@Rahbaran
Copy link
Member

Rahbaran commented Sep 8, 2015

Same here. Tested on develop branch and feature/course-lesson-title-editable.

@brylie
Copy link
Member Author

brylie commented Sep 9, 2015

Please open a separate issue for this bug. It may have been resolved recently, but lets track it separately.

On 8 September 2015 23:14:01 EEST, Amir Rahbaran notifications@github.com wrote:

Same here. Tested on develop branch and
feature/course-lesson-title-editable.


Reply to this email directly or view it on GitHub:
#109 (comment)

Sent from my Android device with K-9 Mail. Please excuse my brevity.

@sebbel
Copy link
Contributor

sebbel commented Sep 10, 2015

Opened #113

@brylie
Copy link
Member Author

brylie commented Sep 11, 2015

See PR #114 for a resolution to this issue and #113.

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

No branches or pull requests

3 participants