care_fe icon indicating copy to clipboard operation
care_fe copied to clipboard

Fix Incorrect error message for invalid QR code

Open AnkurPrabhu opened this issue 10 months ago • 12 comments

Proposed Changes

  • Fixes #7571
  • not calling the backend api when the QR code is invalid
  • stop infinite loading screen when the QR code is invalid

@coronasafe/care-fe-code-reviewers @coronasafe/code-reviewers

Merge Checklist

  • [ ] Add specs that demonstrate bug / test a new feature.
  • [ ] Update product documentation.
  • [ ] Ensure that UI text is kept in I18n files.
  • [x] Prep screenshot or demo video for changelog entry, and attach it to issue.
  • [ ] Request for Peer Reviews
  • [ ] Completion of QA

SS: Screenshot 2024-04-12 at 1 36 44 AM

AnkurPrabhu avatar Apr 11 '24 20:04 AnkurPrabhu

@AnkurPrabhu is attempting to deploy a commit to the Open Healthcare Network Team on Vercel.

A member of the Team first needs to authorize it.

vercel[bot] avatar Apr 11 '24 20:04 vercel[bot]

Deploy Preview for care-egov-staging ready!

Name Link
Latest commit 25ab42a25c3cbdb735ae98ff62fcf0e976b04f74
Latest deploy log https://app.netlify.com/sites/care-egov-staging/deploys/66aa47af280a2700085a1280
Deploy Preview https://deploy-preview-7588--care-egov-staging.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

netlify[bot] avatar Apr 11 '24 20:04 netlify[bot]

@bodhish i have updated the pr

AnkurPrabhu avatar Apr 17 '24 14:04 AnkurPrabhu

Refer: https://github.com/coronasafe/care_fe/issues/4467

rithviknishad avatar Apr 21 '24 16:04 rithviknishad

Hi, This pr has been automatically marked as stale because it has not had any recent activity. It will be automatically closed if no further activity occurs for 7 more days. Thank you for your contributions.

github-actions[bot] avatar Apr 29 '24 04:04 github-actions[bot]

:wave: Hi, @AnkurPrabhu, Conflicts have been detected against the base branch. Please rebase your branch against the base branch.


This message is automatically generated by prince-chrismc/label-merge-conflicts-action so don't hesitate to report issues/improvements there.

github-actions[bot] avatar May 06 '24 15:05 github-actions[bot]

@AnkurPrabhu clear the merge conflict

CC: @rithviknishad

nihal467 avatar May 07 '24 06:05 nihal467

:wave: Hi, @AnkurPrabhu, Conflicts have been detected against the base branch. Please rebase your branch against the base branch.


This message is automatically generated by prince-chrismc/label-merge-conflicts-action so don't hesitate to report issues/improvements there.

github-actions[bot] avatar May 07 '24 09:05 github-actions[bot]

Hi, This pr has been automatically marked as stale because it has not had any recent activity. It will be automatically closed if no further activity occurs for 7 more days. Thank you for your contributions.

github-actions[bot] avatar May 15 '24 04:05 github-actions[bot]

@nihal467 i have resolved the conflict

AnkurPrabhu avatar May 19 '24 14:05 AnkurPrabhu

@AnkurPrabhu remove the additional "something went wrong...!" error message , when we scan invalid QR

image

nihal467 avatar May 22 '24 11:05 nihal467

@rithviknishad can you tell me if my approach is correct ?

AnkurPrabhu avatar May 22 '24 14:05 AnkurPrabhu

Hi, This pr has been automatically marked as stale because it has not had any recent activity. It will be automatically closed if no further activity occurs for 7 more days. Thank you for your contributions.

github-actions[bot] avatar May 31 '24 04:05 github-actions[bot]

:wave: Hi, @AnkurPrabhu, Conflicts have been detected against the base branch. Please rebase your branch against the base branch.


This message is automatically generated by prince-chrismc/label-merge-conflicts-action so don't hesitate to report issues/improvements there.

github-actions[bot] avatar Jun 05 '24 12:06 github-actions[bot]

LGTM @khavinshankar can you resolve the merge conflict and merge it

nihal467 avatar Jul 31 '24 13:07 nihal467

Deploy Preview for care-net failed.

Name Link
Latest commit d4833c66370fa55f3f3b31e564afe4dc33d8e0be
Latest deploy log https://app.netlify.com/sites/care-net/deploys/66aa3f1c6bb59d000857d6ec

netlify[bot] avatar Jul 31 '24 13:07 netlify[bot]

LGTM

nihal467 avatar Aug 01 '24 07:08 nihal467

@AnkurPrabhu Your efforts have helped advance digital healthcare and TeleICU systems. :rocket: Thank you for taking the time out to make CARE better. We hope you continue to innovate and contribute; your impact is immense! :raised_hands:

github-actions[bot] avatar Aug 01 '24 08:08 github-actions[bot]