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

Refreshing component state not rendering new state #16

Open
yudin-s opened this issue Aug 27, 2020 · 6 comments
Open

Refreshing component state not rendering new state #16

yudin-s opened this issue Aug 27, 2020 · 6 comments

Comments

@yudin-s
Copy link

yudin-s commented Aug 27, 2020

Hello @robtweed I have find and issue with the custom component.

In www/components/ptwq/assembly/ptwq-summary-assembly.js line 95 when I go back to patient summary nothing is happens but by logs in console everything looks correct. Looks like mgWebComponents just not refreshing DOM by regular setState loop.

This code is executing correct once and that's all

@robtweed
Copy link

Where do I find the source code for that assembly module? And what's the URL (and login) for the running app so I can see the problem for myself?

@robtweed
Copy link

Also can you provide me with a list of steps to take in order to replicate the issue and see it for myself?

@yudin-s
Copy link
Author

yudin-s commented Aug 27, 2020

@robtweed this video https://www.loom.com/share/7de85ce4d8ca4f2aba0171c846669186 demonstrates how issue is happens

@yudin-s
Copy link
Author

yudin-s commented Aug 27, 2020

@yudin-s
Copy link
Author

yudin-s commented Aug 27, 2020

Issue in both things are pretty simple for understanding. As I can see with Issue#16 issue based on mgWebComponent dom refresh - we send setState but DOM aren't refreshed as needed

@tony-shannon
Copy link
Member

tony-shannon commented Aug 27, 2020

image
image
so now we want to add a new diagnosis and we do so, by going to the diagnosis page and adding new one
image
new problem /diagnosis added in the master detail table
but not seen when we go back to view the patient summary
image

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

No branches or pull requests

3 participants