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

Update ThirdPartyRelationshipAttribute nomenclature #304

Merged

Conversation

Milena-Czierlinski
Copy link
Contributor

Readiness checklist

  • I added/updated tests.
  • I ensured that the PR title is good enough for the changelog.
  • I labeled the PR.

@Milena-Czierlinski Milena-Czierlinski added wip Work in Progress (blocks mergify from auto update the branch) enhancement New feature or request labels Oct 21, 2024
Copy link

codecov bot commented Oct 21, 2024

Codecov Report

Attention: Patch coverage is 88.38174% with 28 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
...nsumption/src/consumption/ConsumptionCoreErrors.ts 53.33% 7 Missing ⚠️
packages/runtime/src/events/EventProxy.ts 50.00% 6 Missing ⚠️
...ion/src/modules/attributes/AttributesController.ts 69.23% 4 Missing ⚠️
...ationshipAttributeDeletedByPeerNotificationItem.ts 85.18% 4 Missing ⚠️
...n/ThirdPartyRelationshipAttributeSucceededEvent.ts 80.00% 2 Missing ⚠️
...kages/runtime/src/useCases/common/RuntimeErrors.ts 33.33% 2 Missing ⚠️
...ion/src/modules/attributes/local/LocalAttribute.ts 92.85% 1 Missing ⚠️
...AttributeDeletedByPeerNotificationItemProcessor.ts 90.90% 1 Missing ⚠️
...eteThirdPartyRelationshipAttributeAndNotifyPeer.ts 94.73% 1 Missing ⚠️
Files with missing lines Coverage Δ
...nsumption/src/consumption/ConsumptionController.ts 96.73% <100.00%> (ø)
...irdPartyRelationshipAttributeDeletedByPeerEvent.ts 100.00% <100.00%> (ø)
...s/ThirdPartyRelationshipAttributeSucceededEvent.ts 100.00% <100.00%> (ø)
...consumption/src/modules/attributes/events/index.ts 100.00% <100.00%> (ø)
...ges/consumption/src/modules/notifications/index.ts 100.00% <100.00%> (ø)
...readAttribute/ReadAttributeRequestItemProcessor.ts 90.72% <100.00%> (-0.42%) ⬇️
packages/content/src/notifications/items/index.ts 100.00% <100.00%> (ø)
...rtyOwnedRelationshipAttributeDeletedByPeerEvent.ts 100.00% <100.00%> (ø)
...irdPartyRelationshipAttributeDeletedByPeerEvent.ts 100.00% <100.00%> (ø)
packages/runtime/src/events/consumption/index.ts 100.00% <100.00%> (ø)
... and 12 more

@Milena-Czierlinski Milena-Czierlinski changed the title Feature/Update ThirdPartyRelationshipAttribute nomenclature Update ThirdPartyRelationshipAttribute nomenclature Oct 21, 2024
@Milena-Czierlinski Milena-Czierlinski removed the wip Work in Progress (blocks mergify from auto update the branch) label Oct 21, 2024
@Milena-Czierlinski Milena-Czierlinski marked this pull request as ready for review October 21, 2024 15:19
@jkoenig134
Copy link
Member

Do we really need to introduce all these breaking changes now?

@Milena-Czierlinski
Copy link
Contributor Author

@jkoenig134 Should we also keep a deprecated function deleteThirdPartyOwnedRelationshipAttributeAndNotifyPeer in addition to the new function deleteThirdPartyRelationshipAttributeAndNotifyPeer in the AttributesFacade for backwards compatibility?

@jkoenig134
Copy link
Member

If one calls the other one, why not :)

@Milena-Czierlinski Milena-Czierlinski merged commit d2de563 into main Oct 23, 2024
18 checks passed
@Milena-Czierlinski Milena-Czierlinski deleted the feature/third-party-relationship-attribute-nomenclature branch October 23, 2024 14:34
@jkoenig134 jkoenig134 added refactoring Refactoring of code and removed enhancement New feature or request labels Oct 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactoring Refactoring of code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants