Skip to content
This repository has been archived by the owner on Nov 28, 2022. It is now read-only.

🐛 clear date error when closing the PSM #657

Merged

Conversation

kevinansfield
Copy link
Member

closes TryGhost/Ghost#8359

  • if the date/time picker is in an error state when the PSM is closed, reset it to the currently saved time
  • resolves the problem with the flash of an error as the publish menu is closed as it's no longer possible to be in a hidden error state

@kirrg001 kirrg001 self-assigned this Apr 19, 2017
@coveralls
Copy link

Coverage Status

Coverage increased (+0.09%) to 73.273% when pulling 684debd on kevinansfield:publishmenu-psm-improvements into c1a9726 on TryGhost:master.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.09%) to 73.33% when pulling b760a5f on kevinansfield:publishmenu-psm-improvements into ff12348 on TryGhost:master.

Copy link
Contributor

@kirrg001 kirrg001 left a comment

Choose a reason for hiding this comment

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

I can't open the PSM with this branch.
Latest Chrome. Fresh build.
I have tested with master, works on master. So must be related to this branch.

@kirrg001
Copy link
Contributor

Forgot to say: No errors in the debug console.

closes TryGhost/Ghost#8359
- if the date/time picker is in an error state when the PSM is closed, reset it to the currently saved time
- resolves the problem with the flash of an error as the publish menu is closed as it's no longer possible to be in a hidden error state
@kevinansfield
Copy link
Member Author

@kirrg001 are there any particular circumstances in which you aren't able to open the PSM or is it everywhere? New/old posts, only certain posts? I'm not able to replicate to investigate unfortunately

@coveralls
Copy link

Coverage Status

Coverage increased (+0.09%) to 73.314% when pulling abc624e on kevinansfield:publishmenu-psm-improvements into b98a804 on TryGhost:master.

@kirrg001
Copy link
Contributor

Will try again and see if i can share steps to reproduce.

@kirrg001
Copy link
Contributor

I just got this behaviour on my branch #661 as well. So it's not related to this branch.

@kevinansfield
Copy link
Member Author

@kirrg001 are you able to test with this PR merged on top? #662

I think that is likely to be the problem rather than anything in this PR.

Copy link
Contributor

@kirrg001 kirrg001 left a comment

Choose a reason for hiding this comment

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

I have rebased this PR locally with #662.
Works 👍

@kirrg001 kirrg001 merged commit bab4aba into TryGhost:master Apr 24, 2017
@kevinansfield kevinansfield deleted the publishmenu-psm-improvements branch June 4, 2018 10:02
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

"Date must be in the past" error shows up in the publish menu
3 participants