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

check-in breaks the default view for a folder #56

Closed
k2soule opened this issue Sep 18, 2015 · 7 comments · Fixed by #108
Closed

check-in breaks the default view for a folder #56

k2soule opened this issue Sep 18, 2015 · 7 comments · Fixed by #108
Assignees
@paregorios
Labels
bug priority: high priority: urgent review
Milestone
February 2016

Comments

@k2soule
Copy link

k2soule commented Sep 18, 2015

When a folder is set to display a particular content item as its default view, and that item is checked out + checked in (as per our workflow procedure), the folder reverts to "standard" view upon check-in. This creates a serious problem for editors, who have no way to edit a published file that acts as the default view of its container (without making a mess).
@paregorios paregorios modified the milestones: Backlog, Sprint 1 Oct 28, 2015
@paregorios paregorios modified the milestone: November Nov 17, 2015
@paregorios paregorios removed their assignment Nov 17, 2015
@cguardia cguardia self-assigned this Nov 20, 2015
@paregorios paregorios modified the milestones: November, December Dec 1, 2015
alecpm added a commit that referenced this issue Dec 16, 2015
@alecpm
Replace description with subtitle. Refs #55, #56
7ed28df
@paregorios paregorios modified the milestones: December, January Jan 13, 2016
@paregorios paregorios assigned paregorios and unassigned cguardia Jan 13, 2016
@cewing
Copy link

cewing commented Jan 15, 2016

@paregorios I can verify that the same behavior happens with a default installation of Plone. I am investigating to discover why, but it appears to be an issue in the package that provides working copy support. I'll keep you posted.

cewing added a commit that referenced this issue Jan 15, 2016
@cewing
update pin for p.a.iterate in order to fix #56
904636a
@cewing cewing mentioned this issue Jan 15, 2016
Merged
@cewing
Copy link

cewing commented Jan 15, 2016

@alecpm please mark reviewing after this is merged and deployed, and assign to @paregorios or @k2soule so they can verify the fix.

@cewing cewing assigned alecpm and unassigned cewing Jan 15, 2016
alecpm added a commit that referenced this issue Jan 16, 2016
@alecpm
Merge pull request #108 from /issues/56-default-item-broken-on-checkin
ef0b38b 
Fix default view item lost on checkin
@alecpm alecpm reopened this Jan 16, 2016
@alecpm
Copy link

alecpm commented Jan 16, 2016

Didn't mean to close the issue, but it is merged and deployed to dev.

@skleinfeldt
Copy link

over to @paregorios for review

@skleinfeldt skleinfeldt assigned paregorios and unassigned alecpm Jan 19, 2016
@paregorios
Copy link
Member

this appears to be working on my local instance

@paregorios paregorios modified the milestones: January, February 2016 Feb 5, 2016
@paregorios
Copy link
Member

This is indeed fixed. Closing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@paregorios paregorios

Labels
bug priority: high priority: urgent review
Projects
None yet
Milestone
February 2016
Development

Successfully merging a pull request may close this issue.

Fix default view item lost on checkin isawnyu/isaw.web
6 participants
@alecpm @paregorios @cguardia @cewing @skleinfeldt @k2soule