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

BAH-3053 | add pre-patient attribute in registration page (#600) #638

Merged
merged 23 commits into from
Apr 22, 2024

Conversation

kavitha-sundararajan
Copy link
Contributor

@kavitha-sundararajan kavitha-sundararajan commented Jul 12, 2023

  • BAH-3053 | add pre-patient attribute in registration page

  • BAH-3053 | added check for pre-registration attribute

  • BAH-3053 | refactored to drive by single config

JIRA Ref : BAH-3053

  • Sends additional search param to be filtered out from lucene search based on config

(PR : #639)
JIRA ref : BAH-3065

kavitha-sundararajan and others added 21 commits July 13, 2023 00:20
* BAH-3053 | add pre-patient attribute in registration page

* BAH-3053 | added check for pre-registration attribute

* BAH-3053 | refactored to drive by single config
"PATIENT_ATTRIBUTE_MIDDLE_NAME_LOCAL": "Middle Name in Arabic",
"PRE_PATIENT_LABEL": "Patient"
Copy link
Contributor

Choose a reason for hiding this comment

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

@kavitha-sundararajan should we rename this from PRE_PATIENT_LABEL possible other names as we discussed.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

updated as discussed @deepthi-mantena

}
}
.pre-patient-label{
width: 14em !important;
Copy link
Contributor

Choose a reason for hiding this comment

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

@kavitha-sundararajan to be consistent it would be great if this naming also can be changed too

Copy link
Contributor Author

Choose a reason for hiding this comment

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

updated as above @deepthi-mantena

@kavitha-sundararajan kavitha-sundararajan merged commit df7510f into master Apr 22, 2024
2 checks passed
@kavitha-sundararajan kavitha-sundararajan deleted the BAH-3053 branch April 22, 2024 08:59
kavitha-sundararajan added a commit that referenced this pull request May 7, 2024
* BAH-3053 | add pre-patient attribute in registration page (#600)

* BAH-3053 | add pre-patient attribute in registration page

* BAH-3053 | added check for pre-registration attribute

* BAH-3053 | refactored to drive by single config

* Kavitha | refactored pre-patient flow based on configs

* Kavitha | BAH-3065 | add additional search param for lucene search (#604)

* Update package.json (#884)

* [Rahul] | BAH-3710 | Fix. Remove Transifex Script

* [Bahmni Infra] | Add. Update Translation Resources

* Parvathy | BAH-2833 | Add. Display Error Message When Document Size Limit Exceed (#875)

* [Rahu] | BAH-3522 | Fix. Improve Test Coverage For DateUtils

* [Bahmni Infra] | Add. Update Translation Resources

* Parvathy | BAH-2833 | Fix. Handle Document Size Limit Error Message

* Revert "Update package.json (#884)"

This reverts commit 30091b3.

* Revert "[Rahul] | BAH-3710 | Fix. Remove Transifex Script"

This reverts commit bbc704d.

* Revert "Parvathy | BAH-2833 | Add. Display Error Message When Document Size Limit Exceed (#875)"

This reverts commit 67a004b.

* Revert "[Rahu] | BAH-3522 | Fix. Improve Test Coverage For DateUtils"

This reverts commit 1bf9ff7.

* Revert "[Bahmni Infra] | Add. Update Translation Resources"

This reverts commit 95d96c4.

* Revert "Parvathy | BAH-2833 | Fix. Handle Document Size Limit Error Message"

This reverts commit 95fdfac.

* Kavitha | refactor to accept grouped configs

* Revert unnecessary commit

* BAH-3053 | renamed configuration names

* BAH-3053 | renamed lables to related patient identifier

---------
kavitha-sundararajan added a commit that referenced this pull request May 9, 2024
* BAH-3053 | add pre-patient attribute in registration page (#600) (#638)

* BAH-3053 | add pre-patient attribute in registration page (#600)

* BAH-3053 | add pre-patient attribute in registration page

* BAH-3053 | added check for pre-registration attribute

* BAH-3053 | refactored to drive by single config

* Kavitha | refactored pre-patient flow based on configs

* Kavitha | BAH-3065 | add additional search param for lucene search (#604)

* Update package.json (#884)

* [Rahul] | BAH-3710 | Fix. Remove Transifex Script

* [Bahmni Infra] | Add. Update Translation Resources

* Parvathy | BAH-2833 | Add. Display Error Message When Document Size Limit Exceed (#875)

* [Rahu] | BAH-3522 | Fix. Improve Test Coverage For DateUtils

* [Bahmni Infra] | Add. Update Translation Resources

* Parvathy | BAH-2833 | Fix. Handle Document Size Limit Error Message

* Revert "Update package.json (#884)"

This reverts commit 30091b3.

* Revert "[Rahul] | BAH-3710 | Fix. Remove Transifex Script"

This reverts commit bbc704d.

* Revert "Parvathy | BAH-2833 | Add. Display Error Message When Document Size Limit Exceed (#875)"

This reverts commit 67a004b.

* Revert "[Rahu] | BAH-3522 | Fix. Improve Test Coverage For DateUtils"

This reverts commit 1bf9ff7.

* Revert "[Bahmni Infra] | Add. Update Translation Resources"

This reverts commit 95d96c4.

* Revert "Parvathy | BAH-2833 | Fix. Handle Document Size Limit Error Message"

This reverts commit 95fdfac.

* Kavitha | refactor to accept grouped configs

* Revert unnecessary commit

* BAH-3053 | renamed configuration names

* BAH-3053 | renamed lables to related patient identifier

---------

* BAH-3742 | Add. Custom Patient Attribute Icon

* kid icon refactor

* Kavitha|BAH-3742| refactor for combined icon configs

* Kavitha|fix failed test cases

* BAH-3472| refactor icon config in patient context

* BAH-3742 | fix failed test cases

* [Rahul] | BAH-3742 | Fix. Broken Icon HTML

* [Rahul] | BAH-3742 | Refactor. Remove Whitespace

---------

Co-authored-by: Rahul Ramesh <[email protected]>

* add custom attr for lucene search

* Kavitha|add test cases to meet coverage

---------

Co-authored-by: Rahul Ramesh <[email protected]>
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

Successfully merging this pull request may close these issues.

6 participants