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

EHR. Patient information screen. Changes are not saved #765

Open
YevheniiaKulik opened this issue Jan 23, 2025 · 4 comments
Open

EHR. Patient information screen. Changes are not saved #765

YevheniiaKulik opened this issue Jan 23, 2025 · 4 comments
Assignees
Labels
bug Something isn't working

Comments

@YevheniiaKulik
Copy link
Contributor

YevheniiaKulik commented Jan 23, 2025

Original Creator: @olhakovalenko91
Tested on:

  • ENV [demo], Version [1.0.4]
  • Browser [Chrome]

Steps To Reproduce:

  1. Go to 'Patient information' screen.
  2. Make some changes to patient's info.
  3. Click on [Save changes] button.

Expected behavior
Changes are saved without error in dev console.
Actual behavior
Changes are not saved. Error is displayed on dev console.
Screenshots/Video

image

@YevheniiaKulik YevheniiaKulik added the bug Something isn't working label Jan 23, 2025
@YevheniiaKulik
Copy link
Contributor Author

Comment by @YevheniiaKulik demo 1.0.20

@vatsalparikh issue is still valid

Image

@vatsalparikh
Copy link

Okay, will look into this in the next couple of days

@VladMstv VladMstv assigned RomanGolchuk and unassigned vatsalparikh Feb 4, 2025
@VladMstv VladMstv moved this from Backlog to Todo in Ottehr Product Feb 4, 2025
@RomanGolchuk RomanGolchuk moved this from Todo to In Progress in Ottehr Product Feb 4, 2025
@imbenham
Copy link
Contributor

imbenham commented Feb 12, 2025

@RomanGolchuk It was pointed out to us today on a call with a customer that editing either the email or phone field caused the text to appear in the other field simultaneously. Maybe you can check that behavior while working on this ticket? I'll create a separate ticket but it seems like it would make sense to handle both at once if you're already working on this ticket.

Edit: I looked into this and was not able to reproduce the behavior, so I believe it's been fixed. I assigned the ticket to myself and just moved it into ready for testing to confirm it is no longer an issue.

@RomanGolchuk
Copy link
Contributor

Thanks for the update!
I do remember fixing a typo in the Contact Information section where the email and phone selectors were duplicated. However, that was quite a while ago, before the project moved to the public repository.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: In Progress
Development

No branches or pull requests

4 participants