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

HAI-2749 Use data instead of entity for hakemus update change check #852

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

corvidian
Copy link
Contributor

Description

Compare the hakemus update to a HakemusData instead of an ApplicationEntity when checking if the update has any changes.

This allows the same method to be used to check if a täydennys has any changes over the original hakemus or the previously saved täydennys.

Jira Issue: https://helsinkisolutionoffice.atlassian.net/browse/HAI-2749

Type of change

  • Bug fix
  • New feature
  • Other

Instructions for testing

Should work like before.

Checklist:

  • I have written new tests (if applicable)
  • I have ran the tests myself (if applicable)
  • I have made necessary changes to the documentation, link to confluence
    or other location:

@corvidian corvidian force-pushed the HAI-2749/compare-hakemus-update-changes-against-hakemusdata branch from 90e7eed to 4a8f392 Compare October 21, 2024 07:10
Compare the hakemus update to a ` HakemusData` instead of an
`ApplicationEntity` when checking if the update has any changes.

This allows the same method to be used to check if a täydennys has any
changes over the original hakemus or the previously saved täydennys.
@corvidian corvidian force-pushed the HAI-2749/compare-hakemus-update-changes-against-hakemusdata branch from 4a8f392 to b949e26 Compare October 22, 2024 06:47
Copy link

sonarcloud bot commented Oct 22, 2024

Quality Gate Failed Quality Gate failed

Failed conditions
62.8% Coverage on New Code (required ≥ 65%)

See analysis details on SonarCloud

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.

1 participant