Skip to content

Fixing that end date is required when creating single day event

Timo Hartmann requested to merge 277-event_bug_fix_enddate_required into master

Closes #277 (closed)

What does this MR do?

When closing issue #277 (closed), I accidentally removed a line of code which resulted in the following bug: the end date field is now required when creating a single-day event. I pasted the missing line back in, which should fix the current behavior.

How confident are you it won't break things if deployed?

Pretty sure, since I know that creating single-day events worked before I had deleted the respective line of code.

Links to related issues

Checklist

  • added a test, or explain why one is not needed/possible... --> not needed since I only re-inserted a line of code that I had removed accidentally
  • no unrelated changes
  • asked someone for a code review
  • joined #foodsharing-beta channel at https://slackin.yunity.org
  • added an entry to CHANGELOG.md (description, merge request link, username(s)) --> no changelog entry needed since it is only beta
  • Once your MR has been merged, you are responsible to update the #foodsharing-beta Slack channel about what has been changed here. They will test your work in different browsers, roles or other settings
Edited by Chris Oelmueller

Merge request reports