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

full-stop: I can put a stop in where there already is one #4

Open
rabsef-bicrym opened this issue Jan 28, 2023 · 4 comments
Open

full-stop: I can put a stop in where there already is one #4

rabsef-bicrym opened this issue Jan 28, 2023 · 4 comments
Assignees

Comments

@rabsef-bicrym
Copy link
Member

rabsef-bicrym commented Jan 28, 2023

image
The period starting on the 21st was closed on the 28th prior to entering the closure on the 26th. The value of the 26th sticks, losing the 28th.

@rabsef-bicrym
Copy link
Member Author

Let's just preclude this behavior for now - we need a better way of identifying what pokes you'd need to issue to wipe out everything in a range and poke those down. that would be idea. For now, just make stopping a period permanent and not replaceable in this way.

@rabsef-bicrym rabsef-bicrym added this to the Grant Milestone 1 milestone Jan 28, 2023
@sink666
Copy link
Collaborator

sink666 commented Jan 29, 2023

Making permament, as in, once you've issued a stop on an open period the option in the menu is perhaps removed?

@rabsef-bicrym
Copy link
Member Author

Yes... I think? Perhaps this is an open question to the community.

@sink666
Copy link
Collaborator

sink666 commented Jan 31, 2023

See #9 for an implementation of something like this. Once a stop or start is logged, the menu changes to remove the options. Specifically on highlighting a day with a stop or start you get the option to remove it. The Spot and Rate options are removed on days where you shouldn't be able to do them.

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

No branches or pull requests

2 participants