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

Setting an event/location to draft noes not change older versions #1497

Closed
timobrembeck opened this issue May 25, 2022 · 0 comments · Fixed by #1555
Closed

Setting an event/location to draft noes not change older versions #1497

timobrembeck opened this issue May 25, 2022 · 0 comments · Fixed by #1555
Assignees
Labels
‼️ prio: high Needs to be resolved ASAP. 🐛 bug Something isn't working ☺️ effort: low Should be doable in <4h
Milestone

Comments

@timobrembeck
Copy link
Member

timobrembeck commented May 25, 2022

Describe the Bug

When an event/location is saved as draft, only the latest version is marked as "draft" and older public versions are still delivered via the API

Steps to Reproduce

  1. Publish event
  2. Change the content and save as draft

Expected Behavior

The event should not be delivered at all - since the status is now "draft"

Actual Behavior

The old published version is delivered

Additional Information

We already solved this for pages (all past versions need to be stored as draft when an event is stored as draft) and also need to fix this for events and pois.
Since changing all past versions to draft also means that users without publishing permissions could un-publish events/locations, we have to repeat the fixes for #1445 for events/locations as well.

@timobrembeck timobrembeck added 🐛 bug Something isn't working ‼️ prio: high Needs to be resolved ASAP. ☺️ effort: low Should be doable in <4h labels May 25, 2022
@timobrembeck timobrembeck added this to the Version 1.2 milestone May 25, 2022
@MizukiTemma MizukiTemma self-assigned this Jun 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
‼️ prio: high Needs to be resolved ASAP. 🐛 bug Something isn't working ☺️ effort: low Should be doable in <4h
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants