-
Notifications
You must be signed in to change notification settings - Fork 42
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
Remove rules in webapp #1047
Comments
@arjunk do watch this card. The programSummaryRule and encounterEligibilityRule are currently bringing down production. |
@vinayvenu looks like we cant remove eligiblePrograms and eligibleProgramEncounters calls since it is based on them we are displaying the programs and encounter options that an Data entry person can click on to enter. We can remove programSummaryRule though since mostly its data will be available in program enrolment or in encounters. @arjunk @nupoorkhandelwal @salilsamanvay what will be the impact on ground if we remove the programSummaryRule from DEA. We discussed that since the purpose of DEA is to facilitate data entry, we can remove the summary rule. We can think of removing subjectSummary as well. Or is the naming 'Data Entry App' incorrect and its been used for other purposes also? Based on that we can decide on if we want to make the removal permanent or temporary until the other performance fixes come in. |
closing since duplicate: #1042 |
Currently the server is under more load than it can handle. Call to program Summary rule takes up 8%.
So we can get rid of programSummaryRule from DEA since DEA's purpose is data entry.
The text was updated successfully, but these errors were encountered: