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

End date of recurring events incorrectly set to initial ocurrence in events API #2008

Closed
timobrembeck opened this issue Jan 16, 2023 · 0 comments · Fixed by #2030
Closed
Assignees
Labels
🐛 bug Something isn't working ☺️ effort: low Should be doable in <4h 🆘 prio: urgent Needs to be resolved now(?)
Milestone

Comments

@timobrembeck
Copy link
Member

timobrembeck commented Jan 16, 2023

Describe the Bug

The end date of recurring events is incorrectly set to the end date of the initial occurrence of the event in the API response.

Steps to Reproduce

  1. Retrieve events API http://localhost:8000/api/augsburg/de/events/
  2. See incorrect date

Expected Behavior

The end date should contain the end date of the specific occurrence

Actual Behavior

The end date of the initial even is used

Additional Information

This bug was introduced with commit 4465f01 and is also reflected in our current test data:

"start": "2030-01-03T13:00:00+01:00",
"start_date": "2030-01-03",
"start_time": "13:00:00",
"end": "2030-01-01T15:00:00+01:00",
"end_date": "2030-01-01",
"end_time": "15:00:00",

@timobrembeck timobrembeck added 🐛 bug Something isn't working ☺️ effort: low Should be doable in <4h 🆘 prio: urgent Needs to be resolved now(?) labels Jan 16, 2023
@timobrembeck timobrembeck added this to the 23Q1 milestone Jan 16, 2023
@MizukiTemma MizukiTemma self-assigned this Jan 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 bug Something isn't working ☺️ effort: low Should be doable in <4h 🆘 prio: urgent Needs to be resolved now(?)
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants