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

MOSIP-28646 code changes (#1757) #1767

Closed
wants to merge 44 commits into from

Conversation

sowmya695
Copy link
Contributor

No description provided.

biometricModalitySegmentsMap = biometricModalitySegmentsMapforAgeGroup.get(ageGroup);
}
else {
biometricModalitySegmentsMap = biometricModalitySegmentsMapforAgeGroup.get("DEFAULT");
Copy link
Member

Choose a reason for hiding this comment

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

if the map does not contain default age group, how are we handling that ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

For Else condition we assumed always DEFAULT should be present and proceeded without adding any null check.

Copy link
Member

Choose a reason for hiding this comment

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

As discussed, @sowmya695 to create a Jira ticket to handle this case.

@sowmya695 sowmya695 force-pushed the MOSIP-28646_develop branch 2 times, most recently from 7c82838 to 2ef8a29 Compare February 8, 2024 06:50
sowmya695 and others added 27 commits February 8, 2024 12:20
Co-authored-by: Sowmya Ujjappa Banakar <[email protected]>
Signed-off-by: Sowmya Ujjappa Banakar <[email protected]>
Signed-off-by: Sowmya Ujjappa Banakar <[email protected]>
MOSIP-28445 Removed DB health check for camel

Signed-off-by: Sowmya Ujjappa Banakar <[email protected]>
Co-authored-by: Sowmya Ujjappa Banakar <[email protected]>
Signed-off-by: Sowmya Ujjappa Banakar <[email protected]>
Signed-off-by: Sowmya Ujjappa Banakar <[email protected]>
Signed-off-by: Sowmya Ujjappa Banakar <[email protected]>
Signed-off-by: Sowmya Ujjappa Banakar <[email protected]>
* [MOSIP-29167]added reusable workflow

* [MOSIP-29167]added reusable workflow

* [MOSIP-29167]added reusable workflow

* [MOSIP-29167]added reusable workflow

* [MOSIP-29167]added reusable workflow

* [MOSIP-29167]added reusable workflow

Signed-off-by: Sowmya Ujjappa Banakar <[email protected]>
Co-authored-by: Sowmya Ujjappa Banakar <[email protected]>
Signed-off-by: Sowmya Ujjappa Banakar <[email protected]>
Co-authored-by: Sowmya Ujjappa Banakar <[email protected]>
Signed-off-by: Sowmya Ujjappa Banakar <[email protected]>
Signed-off-by: Sowmya Ujjappa Banakar <[email protected]>
Signed-off-by: Sowmya Ujjappa Banakar <[email protected]>
Signed-off-by: Sowmya Ujjappa Banakar <[email protected]>
Signed-off-by: Sowmya Ujjappa Banakar <[email protected]>
…time from metadata of the packet

Signed-off-by: khuddus shariff <[email protected]>
Signed-off-by: Sowmya Ujjappa Banakar <[email protected]>
…cket date and time from metadata of the packet

Signed-off-by: khuddus shariff <[email protected]>
Signed-off-by: Sowmya Ujjappa Banakar <[email protected]>
…cket date and time from metadata of the packet

Signed-off-by: khuddus shariff <[email protected]>
Signed-off-by: Sowmya Ujjappa Banakar <[email protected]>
)

* MOSIP-28185 registration confirmation notification corrected

Signed-off-by: Sowmya Ujjappa Banakar <[email protected]>

* MOSIP-28185 code fix for packet validator

Signed-off-by: Sowmya Ujjappa Banakar <[email protected]>

---------

Signed-off-by: Sowmya Ujjappa Banakar <[email protected]>
Signed-off-by: Sowmya Ujjappa Banakar <[email protected]>
Co-authored-by: Sowmya Ujjappa Banakar <[email protected]>
Co-authored-by: Sowmya Ujjappa Banakar <[email protected]>
Signed-off-by: Sowmya Ujjappa Banakar <[email protected]>
Signed-off-by: Rakshithb1 <[email protected]>
Signed-off-by: Sowmya Ujjappa Banakar <[email protected]>
Signed-off-by: Monobikash Das <[email protected]>
Signed-off-by: Sowmya Ujjappa Banakar <[email protected]>
Signed-off-by: Monobikash Das <[email protected]>
Signed-off-by: Sowmya Ujjappa Banakar <[email protected]>
Signed-off-by: Monobikash Das <[email protected]>
Signed-off-by: Sowmya Ujjappa Banakar <[email protected]>
Signed-off-by: Monobikash Das <[email protected]>
Signed-off-by: Sowmya Ujjappa Banakar <[email protected]>
Signed-off-by: Monobikash Das <[email protected]>
Signed-off-by: Sowmya Ujjappa Banakar <[email protected]>
Signed-off-by: Monobikash Das <[email protected]>
Signed-off-by: Sowmya Ujjappa Banakar <[email protected]>
…time from metadata of the packet (mosip#1795)

Signed-off-by: khuddus shariff <[email protected]>
Signed-off-by: Sowmya Ujjappa Banakar <[email protected]>
…ger.yml file (mosip#1809)

Signed-off-by: techno-467 <[email protected]>
Signed-off-by: Sowmya Ujjappa Banakar <[email protected]>
sowmya695 and others added 16 commits February 8, 2024 12:20
* MOSIP-29935 Code changes

Signed-off-by: Sowmya Ujjappa Banakar <[email protected]>

* MOSIP-29935 intial changes after new approach

Signed-off-by: Sowmya Ujjappa Banakar <[email protected]>

* MOSIP-29935 code fix

Signed-off-by: Sowmya Ujjappa Banakar <[email protected]>

* MOSIP-29935 code fix

Signed-off-by: Sowmya Ujjappa Banakar <[email protected]>

* MOSIp-29935 code fix after testing

Signed-off-by: Sowmya Ujjappa Banakar <[email protected]>

---------

Signed-off-by: Sowmya Ujjappa Banakar <[email protected]>
Co-authored-by: Sowmya Ujjappa Banakar <[email protected]>
Signed-off-by: Sowmya Ujjappa Banakar <[email protected]>
Signed-off-by: Rakshith B <[email protected]>
Signed-off-by: Sowmya Ujjappa Banakar <[email protected]>
…equestor

Signed-off-by: Monobikash Das <[email protected]>
Signed-off-by: Sowmya Ujjappa Banakar <[email protected]>
Signed-off-by: Sowmya Ujjappa Banakar <[email protected]>
Co-authored-by: Sowmya Ujjappa Banakar <[email protected]>
Signed-off-by: Sowmya Ujjappa Banakar <[email protected]>
…records which are not processed

Signed-off-by: khuddus shariff <[email protected]>
Signed-off-by: Sowmya Ujjappa Banakar <[email protected]>
Signed-off-by: khuddus shariff <[email protected]>
Signed-off-by: Sowmya Ujjappa Banakar <[email protected]>
Signed-off-by: khuddus shariff <[email protected]>
Signed-off-by: Sowmya Ujjappa Banakar <[email protected]>
… to be change

Signed-off-by: khuddus shariff <[email protected]>
Signed-off-by: Sowmya Ujjappa Banakar <[email protected]>
…osip/registration/processor/core/status/util/StatusUtil.java

Signed-off-by: Sowmya Ujjappa Banakar <[email protected]>
Signed-off-by: khuddus shariff <[email protected]>
Signed-off-by: Sowmya Ujjappa Banakar <[email protected]>
Signed-off-by: Monobikash Das <[email protected]>
Signed-off-by: Sowmya Ujjappa Banakar <[email protected]>
* [MOSIP-29167]added reusable workflow

* [MOSIP-29167]added reusable workflow

* [MOSIP-29167]added reusable workflow

* [MOSIP-29167]added reusable workflow

* [MOSIP-29167]added reusable workflow

* [MOSIP-29167]added reusable workflow

Signed-off-by: Sowmya Ujjappa Banakar <[email protected]>
…cket date and time from metadata of the packet

Signed-off-by: khuddus shariff <[email protected]>
Signed-off-by: Sowmya Ujjappa Banakar <[email protected]>
Signed-off-by: Monobikash Das <[email protected]>
Signed-off-by: Sowmya Ujjappa Banakar <[email protected]>
Signed-off-by: Monobikash Das <[email protected]>
Signed-off-by: Sowmya Ujjappa Banakar <[email protected]>
…ger.yml file (mosip#1809)

Signed-off-by: techno-467 <[email protected]>
Signed-off-by: Sowmya Ujjappa Banakar <[email protected]>
@sowmya695 sowmya695 force-pushed the MOSIP-28646_develop branch from 2ef8a29 to bb824b9 Compare February 8, 2024 06:51
Signed-off-by: Sowmya Ujjappa Banakar <[email protected]>
@sowmya695
Copy link
Contributor Author

closing and reopen another pr to solve sign off,https://github.com/mosip/registration/pull/1845/files
because sign off causing lot of conflicts

@sowmya695 sowmya695 closed this Feb 8, 2024
vishwa-vyom added a commit that referenced this pull request Feb 8, 2024
MOSIP-28646 code changes (#1757) refer previous pr #1767
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.

7 participants