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: fixing date parse when there is error in date conditions M2-8275 #1983

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

felipeMetaLab
Copy link
Contributor

📝 Description

🔗 Jira Ticket M2-8275
this PR was created to fix date parse when there is error in date conditions.

📸 Screenshots

Error (dev branch):
https://github.com/user-attachments/assets/4ab6ed60-6486-4b9e-bffc-00d3fa4204f7
https://github.com/user-attachments/assets/0d64e085-4024-4e36-aa08-0a727f1376d4

Solved:
https://github.com/user-attachments/assets/37882507-e5d6-461b-8e8a-4aba62f1e3fd
https://github.com/user-attachments/assets/1bbb4358-ea11-4cc1-a19a-036edce3c3e7

🪤 Peer Testing

1 - launch the admin app.
2 - create or edit an activity into an applet
3 - create a date item
4 - and create conditional logic
5 - set the date and save then click in activity ( check if it will retrieve error or not)

✏️ Notes

for more information, check the the ticket, it contain a video with error or check the video I left on PR, the first 2 videos is retrieving error.

@felipeMetaLab felipeMetaLab changed the title fix/adding value to date pickers, watching dates to parse and parsing… fix: fixing date parse when there is error in date conditions M2-8275 Nov 27, 2024
Copy link

This pull request is automatically being deployed by Amplify Hosting (learn more).

Access this pull request here: https://pr-1983.d19gtpld8yi51u.amplifyapp.com

ChaconC
ChaconC previously approved these changes Nov 29, 2024
@ChaconC ChaconC dismissed their stale review December 4, 2024 03:54

QA Fail

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