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

chore(resources): Packages update #688

Merged
merged 28 commits into from
Sep 24, 2024
Merged

Conversation

Jonathanio123
Copy link
Collaborator

@Jonathanio123 Jonathanio123 commented Sep 2, 2024

  • New feature
  • Bug fix
  • High impact

Description of work:
AB#55718

This updates all packages to the latest version. excluding Fusion.AspNetCore. Fusion.AspNetCore version above 7.0.1 causes the PatchModelBinder to fail to parse PatchProperty<ApiPropertiesCollection?> properties in patch requests.

Where PatchProperty with ApiPropertiesCollection will always be empty. I have created an issue for this: https://github.com/equinor/fusion/issues/400

Issue has been closed and I've added the newest Fusion.AspNetCore version to this PR

Testing:

  • Can be tested
  • Automatic tests created / updated
  • Local tests are passing

Checklist:

  • Considered automated tests
  • Considered updating specification / documentation
  • Considered work items
  • Considered security
  • Performed developer testing
  • Checklist finalized / ready for review

@Jonathanio123 Jonathanio123 marked this pull request as ready for review September 2, 2024 13:29
@Jonathanio123
Copy link
Collaborator Author

Trying to find out why snyk errors out

@Jonathanio123 Jonathanio123 force-pushed the chore/package-update-230824 branch from c814b38 to c1abb22 Compare September 9, 2024 14:35
@Jonathanio123 Jonathanio123 changed the title Packages update chore(resources): Packages update Sep 12, 2024
@Jonathanio123 Jonathanio123 merged commit 12c0596 into master Sep 24, 2024
15 checks passed
@Jonathanio123 Jonathanio123 deleted the chore/package-update-230824 branch September 24, 2024 07:40
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.

2 participants