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

Fix user cannot enter future date for DateSampled when sampling workflow is enabled #2402

Merged
merged 12 commits into from
Oct 11, 2023

Conversation

xispa
Copy link
Member

@xispa xispa commented Oct 9, 2023

Description of the issue/feature this PR addresses

Important

Merge #2401 first

This Pull Request ensures the Date Sampled set by the user either in sample edit form or when registering a sample is within expected date interval.

Current behavior before PR

When sampling workflow is enabled, user can enter past dates in Date Sampled (while Sampling Date remains in future)

Desired behavior after PR is merged

When sampling workflow is enabled, user cannot enter past dates in Date Sampled. Likewise, when sampling workflow is disabled, user cannot enter future dates in Date Sampled.

--
I confirm I have tested this PR thoroughly and coded it according to PEP8
and Plone's Python styleguide standards.

@xispa xispa changed the title Fix DateSampled min and max dates Fix user can enter past Date Sampled when sampling workflow is enabled Oct 9, 2023
@xispa xispa requested a review from ramonski October 9, 2023 17:07
@xispa xispa added the Bug 🐞 label Oct 9, 2023
@xispa xispa closed this Oct 10, 2023
@xispa xispa reopened this Oct 10, 2023
@xispa xispa changed the title Fix user can enter past Date Sampled when sampling workflow is enabled Fix user cannot enter future date for DateSampled when sampling workflow is enabled Oct 10, 2023
@ramonski ramonski merged commit 097676c into 2.x Oct 11, 2023
2 checks passed
@ramonski ramonski deleted the date-sampled-max branch October 11, 2023 18:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

2 participants