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.

Sign up for GitHub

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

Jump to bottom

Page parts title missing on newly updated website #3010

Closed
bricesanchez opened this issue Jul 20, 2015 · 1 comment
Closed

Page parts title missing on newly updated website #3010

bricesanchez opened this issue Jul 20, 2015 · 1 comment

Comments

@bricesanchez
Copy link
Member

Since the merge of #2875, if you update an existing website on refinerycms 3.0 master "hide_sections" option on render_content_page doesn't works.

It's because hide_sections tries to hide a part title and not a part slug.

i'm searching how to fix that. There is no data lose.
@bricesanchez
Copy link
Member Author

The problem is here :

refinerycms/pages/app/presenters/refinery/pages/page_part_section_presenter.rb

Line 9 in 30a0b74

self.id = convert_title_to_id(page_part.title) if page_part.title

I have not update this Presenter. I've to update all page.parts slug and title too.

Fix is in progress.

@bricesanchez bricesanchez mentioned this issue Jul 20, 2015
Closed
bricesanchez pushed a commit that referenced this issue Jul 23, 2015
@parndt
Update page part section presenter to use page part slug as id
3b3e59a 
Add migration to update all page parts slug and title

Fixes #3010
Resolves #3011
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@bricesanchez