Create calendar react-play

[vincentBCP]

First thing, PLEASE READ THIS: ReactPlay Code Review Checklist

Description

A simple calendar app to add, update, and delete events.

Dependecies:

• date-fns
• lodash

Fixes # (issue) #199

Type of change

Please delete options that are not relevant.

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)
• This change requires a documentation update

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce.

Checklist:

• I have performed a self-review of my own code
• I have commented my code, particularly in hard-to-understand areas
• I have made corresponding changes to the documentation
• My changes generate no new warnings
• I have added tests that prove my fix is effective or that my feature works
• New and existing unit tests pass locally with my changes
• Any dependent changes have been merged and published in downstream modules

[vercel]

Someone is attempting to deploy a commit to a Personal Account owned by @atapas on Vercel.

@atapas first needs to authorize it.

[vincentBCP]

Review ready

[atapas]

@vincentBCP kindly edit and update the PR description.

[vincentBCP]

@vincentBCP kindly edit and update the PR description.

Updated the PR description.

[atapas]

@vincentBCP kindly edit and update the PR description.

Updated the PR description.

Thanks @vincentBCP. You also need to create the issue for this play. I have done it: #199

Could you please check and add a bit about the play there? Thanks. We will start the review shortly.

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Updated
react-play	✅ Ready (Inspect)	Visit Preview	May 22, 2022 at 2:15PM (UTC)

[atapas]

@vincentBCP It is a cool play. Just played around with the preview URL. Looks awesome.

I have provided my initial review comments. @koustov will jump into doing the reviews of it as well.

One thing I noticed is, that the UI breaks a bit on mobile. If you can take care of the responsiveness as well, will be great.

[vincentBCP]

Thanks for the review @atapas . I'll work on the suggestions and issues.

[vincentBCP]

Review ready

[atapas]

@vincentBCP Would you take up validation for cases like this? Like start date is ahead of the end date?

[vincentBCP]

@vincentBCP Would you take up validation for cases like this? Like start date is ahead of the end date?

Sure. I'll add this validation.

[vincentBCP]

@vincentBCP Would you take up validation for cases like this? Like start date is ahead of the end date?

Sure. I'll add this validation.

@atapas

I already pushed the changes for time validation, endTime should not be before the startTime now.
I just want to clarify, because I also got confused. The screenshot is correct the event is May 01 from 12:22 PM (noon) to 11:22 PM (evening). Tho, time range should be within the 24hr of the selected (date) only, because there is no feature implemented to create event the spans multiple days.

[vincentBCP]

Thanks for the review @koustov . I'll be working on request changes.

[vincentBCP]

Review ready

[koustov]

LGTM

[atapas]

LGTM

[atapas]

@vincentBCP Please resolve the merge conflicts

[vincentBCP]

Resolved merge conflicts.

[koustov]

LGTM

[atapas]

@all-contributors please add @vincentBCP for Code

[allcontributors]

@atapas

I've put up a pull request to add @vincentBCP! 🎉