Skip to content

Conversation

@anaclos
Copy link

@anaclos anaclos commented Feb 15, 2025

No description provided.

Copy link

@chrisjamiecarter chrisjamiecarter left a comment

Choose a reason for hiding this comment

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

Hey @anaclos 👋,

Excellent work on your Exercise Tracker project submission 🎉 !

I have performed a peer review. Review/ignore any comments as you wish.


🟢 Requirements

⭐ You have fulfilled all of the project requirements!


I will go ahead and mark as approved, keep up the excellent work on the next projects! 😊

Best regards,
@chrisjamiecarter 👍

{
TimeSpan timeSpan = end - start;
double eightHours = 3600 * 8;
return timeSpan.TotalSeconds > 0 && timeSpan.TotalSeconds <= eightHours ? true : false;

Choose a reason for hiding this comment

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

🟢 Validation

😆 Less than 8 hours! Very optimistic, love it!

return dateTime;
}

public static bool OrderValidator(Exercise lastShift, DateTime newStartTime)

Choose a reason for hiding this comment

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

🟠 Shift

❓ Shift or Exercise?

DateTime startTime;
DateTime endTime;

_userInput.ShowMessage("The new shift must be after the last shift interval. The shift interval must not exceed 8 hours. ", "blue");

Choose a reason for hiding this comment

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

🟠 Shift

❓ Shift?

@chrisjamiecarter
Copy link

@anaclos P.S., Please can you close this PR?

@anaclos
Copy link
Author

anaclos commented Mar 7, 2025 via email

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