-
Notifications
You must be signed in to change notification settings - Fork 72
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
Ocrvs 4046 #7744
Open
cleverclope
wants to merge
154
commits into
opencrvs:develop
Choose a base branch
from
cleverclope:ocrvs-4046
base: develop
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Ocrvs 4046 #7744
+3,038
−2,574
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
cleverclope
commented
Oct 8, 2024
- Added open dialog on delete support document
* Add a util function to exclude record corrected history from fhir bundle response * Implement the util method to remove record corrected history from root resolvers * Revert "Implement the util method to remove record corrected history from root resolvers" This reverts commit fe1d6f5. * Refactor exclude record corrected history * Revert "Refactor exclude record corrected history" This reverts commit 588f9c8. * Create and implement a custom filtering predicate to remove corrected history * Remove unused utils * Remove unused imports * Remove check for rejected action as there would be no corrected task after rejection --------- Co-authored-by: Riku Rouvila <riku.rouvila@gmail.com>
…correction request (opencrvs#6981) * review button role status updated for national registrar * updated changelog.md * In search result page, navigate to review correction page if registration status is `CORRECTION_REQUESTED` * Update CHANGELOG.md --------- Co-authored-by: tahmidrahman-dsi <tahmid.rahman@dsinnovators.com> Co-authored-by: Tahmid Rahman <42269993+tahmidrahman-dsi@users.noreply.github.com>
* Rename component file names according to the current convention * Rename PhoneNumberVerificationForm -> AuthDetailsVerificationForm - as we also verify email if the user notification method is set 'email' * Fetch config before rendering react app * Amend withRetry function implementation Co-authored-by: Riku Rouvila <riku.rouvila@gmail.com> * Update CHANGELOG.md --------- Co-authored-by: Riku Rouvila <riku.rouvila@gmail.com>
* fix: amend updated record type * chore: add comment and change function name * fix: handle no informant info for in-progress declaration
* fix: update print record page to open in the same tab * chore: add X icon in the print record page * docs: update changelog
* Remove authentication from dashboard queries api endpoints * Update CHANGELOG.md
… in action modal for action CORRECTED (opencrvs#7094)
…6-7068 Cherry pick 7016, 7068
* fix: update correction request task * fix: update snapshot for task output * fix: set correction changes to read from output * fix: amend correction values and type in client
fix: show correction rejection reason in history
* feat: add new handlebars * docs: update CHANGELOG * test: add registered history entry to mock data
* ocrvs-7002 * replaced 'auto' with 'scroll' * changelog.md updated * fix: react issue with exhaustive deps --------- Co-authored-by: Pyry <pyry.rouvila@gmail.com>
…ying to correct, print or issue record (opencrvs#7061) * failed tests fixed * removeing labelformat for informant * failed test fixed, removeing labelformat for informant * remove labelformatterForInformant * removed labelformatter for requesterLabelMapper in ActionDetailsModal.tsx
…ncrvs#7006) * initial fix * updated changelog.md * updated changelog * filtered with isVisibleField * isUploadButtonVisible now boolean
Cherry pick to v1.5.0
…opencrvs#7066) * Fix "Print and issue to groom|bride" is added to a different variable * groom and bride default name fix --------- Co-authored-by: Tameem Bin Haider <tameem.haider@dsinnovators.com>
* Fix URL path by removing leading slash This change ensures that the constructed URL uses a relative path instead of an absolute path, appending 'resendAuthenticationCode' correctly to the base URL 'https://gateway.farajaland-qa.opencrvs.org/auth/'. * chore(changelog) --------- Co-authored-by: naftis <pyry.rouvila@gmail.com>
…pencrvs#7081) * Removed duplicateTrackingId check in createDuplicateTask method * changelog updated
…pencrvs#7038) * Fix issue opencrvs#7017: Check for valid date to handle incomplete marriage declarations Updated logic to use isValidPlainDate function to check date values, ensuring that null or invalid date values do not cause issues. This change allows the retrieval of incomplete marriage declarations even when the date of marriage is missing. * changelog updated
* Fix: Reset pagination to page 1 when location changes Ensure the currentPageNumber is reset to 1 whenever the location parameter changes to avoid issues where no data is found and the table and pagination components disappear. * changelog updated
…1.5.0 Cherry pick to v1.5 branch
* Covert DocumentUploaderWithOption to functional component * Introduce a prop compressImagesToSizeMB to define maximum size of compressed file * Minor refactor * Add a maxSize prop to prevent uploading document bigger than a defined size * Updates to the messages having variable maximum sizes * Update FormFieldGenerator to make document uploader with options configurable from country config forms * Fix typo * Apply suggestions from code review Co-authored-by: Tameem Bin Haider <tameem.haider@dsinnovators.com> * Update CHANGELOG.md
…-7323-11582 fix(v1.5.0): update assignment check for retrieve action
…-7322-21554 fix(v1.5.0): redis hostname
Co-authored-by: Pyry Rouvila <pyry@opencrvs.org>
…-7328-27850 🍒 Merge changes from PR opencrvs#7328 to release-v1.5.0
* fix: use `emailForNotification` in telecom email * fix: added migration to correct practitioner email * fix: createUser test for correct practitioner email * amend: using iterator pattern to optimize memory usage * chore: update changelog
* chore: rename constants related to fieldChange functions * fix: track custom field value changes
…-7315-13764 fix(v1.5.0): email of practitioner
…-7023-8095 🍒 Merge changes from PR opencrvs#7023 to release-v1.5.0
…-7314-9999 fix(v1.5.0): fetch Documents after resolving Encounters
…-7335-30595 fix(v1.5.0): show comment section when there is no comment in record audit modal
…-7340-9200 fix(v1.5.0): sync review button logic in adv search result
* feat: send notification for rejection event * fix: use key for death inProgress notification * chore: use util functions
…-7345-267 feat(v1.5.0): send notification for rejection event
Release 1.5.0
fix: bring back certificate & notification config
jpye-finch
reviewed
Oct 8, 2024
DialogContent, | ||
DialogContentText, | ||
DialogTitle | ||
} from '@mui/material' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We have a Dialog component in our component library
@cleverclope You might have created your branch from master branch instead of develop. I suggest you close this PR and create a new one with only your commit. As you can see this PR is showing 154 commits currently! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.