Skip to content

CAS-532 - Fix start time interval#487

Merged
jbluks merged 6 commits intomainfrom
germanurrustarazu/cas-532
Sep 16, 2022
Merged

CAS-532 - Fix start time interval#487
jbluks merged 6 commits intomainfrom
germanurrustarazu/cas-532

Conversation

@germanurrus
Copy link
Copy Markdown
Contributor

@germanurrus germanurrus commented Sep 16, 2022

To test this PR:

  • update your machine time to be within the last hour of today
  • Create a proposal and select start time of proposal to be tomorrow
  • On drop down time for startTime you'll see first minutes interval of the day removed to meet the one hour requirement between time now and start of proposal.

@linear
Copy link
Copy Markdown

linear Bot commented Sep 16, 2022

CAS-532

@germanurrus germanurrus marked this pull request as ready for review September 16, 2022 18:28
@germanurrus germanurrus changed the title CAS-532 - Fix interval CAS-532 - Fix start time interval Sep 16, 2022
: 0;
const startTimeInterval = HAS_DELAY_ON_START_TIME
? getStartTimeIntervalWithDelay(date, startDateIsToday)
: getStartTimeIngerval(startDateIsToday);
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Spelling mistake: getStartTimeIngerval

@jbluks jbluks merged commit 1794947 into main Sep 16, 2022
@jbluks jbluks deleted the germanurrustarazu/cas-532 branch September 16, 2022 19:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants