Skip to content
This repository has been archived by the owner on Feb 7, 2024. It is now read-only.

Feature/ae 2029 #1044

Merged
merged 3 commits into from
Nov 21, 2023
Merged

Feature/ae 2029 #1044

merged 3 commits into from
Nov 21, 2023

Conversation

Juholei
Copy link
Contributor

@Juholei Juholei commented Nov 15, 2023

No description provided.

@Juholei Juholei force-pushed the feature/AE-2029 branch 6 times, most recently from 65429ee to dff7f25 Compare November 17, 2023 13:18
@Juholei Juholei marked this pull request as ready for review November 17, 2023 13:19
… and sakkopäätös toimenpide

- Henkilosapuoli and yritysosapuoli ids can be same so osapuoli-specific-data could get mixed up between the two if present in the same toimenpide
- Added osapuoli-type to osapuoli-specific-data which is then checked together with the osapuoli-id if the osapuoli is the correct one
Copy link
Contributor

@solita-antti-mottonen solita-antti-mottonen left a comment

Choose a reason for hiding this comment

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

Voisiko osapuoli id:n ja tyypin yhdistää mapiksi tjsp. jotta olisi ilmeisempää että on tekemässä virhettä jos koittaa yksilöidä osapuolen vain id:n perusteella. Lisäksi tarvitsisi passata vain yksi parametri funktioille jotka odottavat osapuoli-id:tä

…apuoli-specific-data

- Makes it clearer that both need to be checked for osapuoli to match
@Juholei Juholei merged commit 4416a54 into develop Nov 21, 2023
4 checks passed
@Juholei Juholei deleted the feature/AE-2029 branch November 21, 2023 07:24
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants