Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Show schedules till they are completed #3236

Merged
merged 1 commit into from
Sep 14, 2018
Merged

Show schedules till they are completed #3236

merged 1 commit into from
Sep 14, 2018

Conversation

Wouter0100
Copy link
Contributor

@Wouter0100 Wouter0100 commented Sep 6, 2018

Currently schedules are shown on the status page when they are in the status upcoming or in progress. But, when the scheduled time is in the past - it's hidden, even when it's still going on and regardless of the completed time entered.

This PR fixes this issue by hiding the maintenance when it's in the status completed, or the completed time has passed.

Currently schedules are shown on the status page when they are in the status upcoming or in progress. But, when the scheduled time is in the past - it's hidden, even when it's still going on.

This PR fixes this issue by hiding the maintenance when it's in the status completed, or the completed time has passed.
@welcome
Copy link

welcome bot commented Sep 6, 2018

Congratulations on opening your first Pull Request, this is a momentous day for you and us! ✨
To help us out, please make sure that you've followed the below:

@jbrooksuk
Copy link
Member

This makes sense.

Thanks for this!

@jbrooksuk jbrooksuk merged commit de172e5 into cachethq:2.4 Sep 14, 2018
@welcome
Copy link

welcome bot commented Sep 14, 2018

Hooray! Your first Pull Request was merged, here's to many more 🚀

@thomasxd24
Copy link
Contributor

This PR + merge made the scheduled maintenace disappears completely, Please revert this commit or fix the problem.

@Wouter0100
Copy link
Contributor Author

Wouter0100 commented Oct 5, 2018

@thomasxd24 are you sure? We're using this in production, which seems to work as promised. Let me double check.

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.

None yet

3 participants