Skip to content

Issue in the latest merged PR #5

@priyeshbitsp

Description

@priyeshbitsp

Hello @Siddharth352 @piyush-eon

There is a minor miss in this #1

Issue: We should not be always setting the flag to false. Because if the user doubles clicks in the same time period, we don't change the days (as useEffect doesn't run again) but we set the flag to false due to which the circular progress comes up and it persists as the flag is never reset to true again.

Change Needed: Condition to set the flag to false

onClick={() => {
setDays(day.value);
if (days !== day.value) setflag(false);
}}

Thanks!

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions