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

Weno eRx fixes #1139

Merged
merged 3 commits into from Oct 22, 2017
Merged

Weno eRx fixes #1139

merged 3 commits into from Oct 22, 2017

Conversation

sjpadgett
Copy link
Sponsor Member

Fixed imports- had to redo drug paid import sql file as old was breaking transaction!
Redid pharmacies import and wrapped in transaction.
Fixed NDC (drug_id) not being saved with prescription or populating json.
Fixed qualifier and provider id syntax for populating transmission json.
Maybe others!

@sjpadgett
Copy link
Sponsor Member Author

@juggernautsei After all this and checking json sent to ensure all elements are populating I still don't get a response from weno server. It just sends (I guess it sends as all I see is whats in console) and that's it. If I just click transmit after sending a script then weno replys with invalid message as expected. What is weno's expected behavior???

@bradymiller
Copy link
Sponsor Member

@sjpadgett , Nice work. Just let me know when you want me to bring this in. -brady

@sjpadgett
Copy link
Sponsor Member Author

@bradymiller @juggernautsei Another update, the main going on's:
Redo of pharmacy import to match recent csv change (really just back to original).
Restyle imports admin and added import of city and state to narrow imported quantities some.
Fix for auto select of xmit checkboxs and disable not active scripts.
Change toggle action on txMark otherwise they are never cleared and always are selected.(code give a better idea)
The biggy here is behavior change of ajax sending to weno server. Trying to send update to UI can cause timing problems so I changed to batching ajax and UI writing using apply and promises. Now appears to always send and receive as expected.
I've tested fairly well but rec someone else from different setup....
Sherwin, I rec changing your prescription status updates to db until after weno gives success otherwise on fail the prescription state is as if the script was a success...
This should now work once weno messaging issue is figured out

<div class="row text-center">
<div class="form-group well col-lg-5 text-center">
<?php echo generate_form_field(array('data_type' => $GLOBALS['state_data_type'], 'list_id' => $GLOBALS['state_list'], 'field_id' => 'state'), ''); ?>
<input type="text" class="form-control" id="form_city" name="form_city" placeholder="Enter City"><br/><br/>
Copy link
Sponsor Member

Choose a reason for hiding this comment

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

xla() of 'Enter City'
(I finally found one :) )

@bradymiller
Copy link
Sponsor Member

Code looks good to me; just found 1 minor translation issue. If this is working for you, then we should bring this in (since otherwise WENO is not functional in the current codebase). Just give me the word, and I'll bring this into the codebase.

@sjpadgett
Copy link
Sponsor Member Author

Yep go ahead i'll finish working with Sherwin to get weno message solved.

@bradymiller bradymiller merged commit a2208ce into openemr:master Oct 22, 2017
@bradymiller
Copy link
Sponsor Member

Just brought into codebase. Thanks for fixing WENO! -brady

@sjpadgett sjpadgett deleted the weno-erx-fixup branch June 9, 2018 17:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants