search

codefordenver / Comrad

Open-source web application for radio stations to manage show schedules, traffic and compliance
ISC License
25 stars 9 forks source link

Fix for Traffic event update dates not saving issue #840

Closed kmid5280 closed 1 year ago

kmid5280 commented 1 year ago

Guidelines

DEVELOPMENT branch on the LEFT
YOUR branch on the RIGHT

Issue

This issue resolves #834

Description

I did two things here. One, on line 267 I changed the value of body.repeat_rule to include a spread operator incorporating the new values. I think that as it was, this line and the lines on 271-273 were overwriting the new values in the request. I wanted to change this to create a copy of the original values and then overwrite those with any new values. Two, I removed the bracket notation from the conditionals creating the start and end dates. Please test this and verify that it works, and whether there are any other issues.

seankwilliams commented 1 year ago

Thank you, @kmid5280 ! I will test this out and merge it in.

seankwilliams commented 1 year ago

@kmid5280 looking good! I'll deploy this.