250: adding a temporary change in reminderController

[kapirath28]

its my first contribution just tried to fix the issue by

-In the addreminder method, before adding a new reminder, ichecked if the Reminder has a valid time in the future.

-removed reminders from the list that have a localTime before the current time (LocalDateTime.now())

(i am a beginner at this let me know about it.)

[linux-foundation-easycla]

• ❌ - login: @kapirath . The commit (8cb514f) is not authorized under a signed CLA. Please click here to be authorized. For further assistance with EasyCLA, please submit a support request ticket.

[kapirath28]

its my first contribution just tried to fix the issue 250

[robmoffat]

#250

Hi @kapirath28,

Thanks for this - why did you close it? I think you were on the right track. Just looking at the Reminder Bot with @vaibhav-db , we think the "check it's in the future" code should be in this class:

https://github.com/finos/spring-bot/blob/spring-bot-master/tools/reminder-bot/src/main/java/org/finos/springbot/tool/reminders/TimeFinder.java

Do you want to have another try?