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

ONI-173: change insuree head projection #95

Merged
merged 1 commit into from
Oct 18, 2023
Merged

Conversation

olewandowski1
Copy link
Contributor

@olewandowski1 olewandowski1 commented Oct 9, 2023

ONI-173

Change:

  • Fill missing fields in head insuree projection. (Fetching all fields except family - fetching family was causing circular dependency between insuree and family - in this case, we can omit fetching family).

@olewandowski1 olewandowski1 requested review from sniedzielski and a user October 9, 2023 08:57
@sonarqubecloud
Copy link

sonarqubecloud bot commented Oct 9, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@olewandowski1 olewandowski1 marked this pull request as ready for review October 9, 2023 08:59
@dragos-dobre dragos-dobre merged commit baa106e into develop Oct 18, 2023
@dragos-dobre dragos-dobre deleted the feature/ONI-173 branch October 18, 2023 13:20
dragos-dobre added a commit that referenced this pull request Oct 18, 2023
* ONI-16: Add insuree address in enquire. (#86)

* ONI-3: Add insuree address in enquire.

* ONI-16: Add insuree address in enquire.

* hotfix: restore insuree creation from family view (#88)

* ONI-95: profile page implementation (#90)

* ONI-95: insuree page profile init

* ONI-95: profile page adjustments

* ONI-95: translation fix

* ONI-95: render insuree profile link on proper setting

* ONI-3: Added insuree status dropdown. (#91)

* ONI-3: Added insuree status dropdown.

* ONI-3: Added status projection and Readme info.

* ONI-3: Code review fixes for insuree status.

* ONI-3: Code review fix for insuree status.

* ONI-3: Insuree status bug fixed.

* ONI-5: Confirmation number required on condition. (#92)

* ONI-5: Confirmation number required on condition.

* ONI-5: Confirmation type number required depending on confirmation type.

* ONI-5: Fix to Family canSave function

* ONI-5: Confirmation number required variable name change.

* ONI-5: Variable name change.

* ONI-95: block view button if user not permitted, align button styles (#94)

* ONI-173: change insuree head projection (#95)

---------

Co-authored-by: wzglinieckisoldevelo <98958634+wzglinieckisoldevelo@users.noreply.github.com>
Co-authored-by: olewandowski1 <109145288+olewandowski1@users.noreply.github.com>
dragos-dobre added a commit that referenced this pull request Oct 18, 2023
* ONI-16: Add insuree address in enquire. (#86)

* ONI-3: Add insuree address in enquire.

* ONI-16: Add insuree address in enquire.

* hotfix: restore insuree creation from family view (#88)

* ONI-95: profile page implementation (#90)

* ONI-95: insuree page profile init

* ONI-95: profile page adjustments

* ONI-95: translation fix

* ONI-95: render insuree profile link on proper setting

* ONI-3: Added insuree status dropdown. (#91)

* ONI-3: Added insuree status dropdown.

* ONI-3: Added status projection and Readme info.

* ONI-3: Code review fixes for insuree status.

* ONI-3: Code review fix for insuree status.

* ONI-3: Insuree status bug fixed.

* ONI-5: Confirmation number required on condition. (#92)

* ONI-5: Confirmation number required on condition.

* ONI-5: Confirmation type number required depending on confirmation type.

* ONI-5: Fix to Family canSave function

* ONI-5: Confirmation number required variable name change.

* ONI-5: Variable name change.

* ONI-95: block view button if user not permitted, align button styles (#94)

* ONI-173: change insuree head projection (#95)

* remove second declaration of formatLocationString

---------

Co-authored-by: wzglinieckisoldevelo <98958634+wzglinieckisoldevelo@users.noreply.github.com>
Co-authored-by: olewandowski1 <109145288+olewandowski1@users.noreply.github.com>
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.

3 participants