Skip to content

Choose time validation, issue #12#33

Open
SagyBle wants to merge 3 commits intomainfrom
chooseTime-validation
Open

Choose time validation, issue #12#33
SagyBle wants to merge 3 commits intomainfrom
chooseTime-validation

Conversation

@SagyBle
Copy link
Copy Markdown
Collaborator

@SagyBle SagyBle commented Apr 18, 2023

Code Reviewer

Related Issue

Please Review

  • [ ]
  1. Choose time inputs are being tested before sent.
  2. Data is sent only after popup confirmation.
  • need to check with product the actual flow happens after confirming the popup. like is there confirmation page afterwards? or is the popup text is correct?


dispatch(
openPopup({
title: "מעולה! פרטייך נשלחו",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

texts here are not correct. should be a text that's asking the user to confirm to post the session request

Comment thread src/components/ChooseTime.jsx Outdated
title: "מעולה! פרטייך נשלחו",
text: "בקרוב ניצור איתך קשר בוואטסאפ ונקשר אותך לשיחה עם אדם עם תחושות שונות בנושא שבחרת",
handleConfirm: () => {
postSessionRequest(session);
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
postSessionRequest(session);
try {
await postSessionRequest(session);
// close popup and redirect to a success page (not back to emotions selection)
} catch(error) {
// handle network request error by displaying an error to the user
}

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.

3 participants