Skip to content

fix: update PolygonPartsProcessPayload to include shouldClearEntities#85

Merged
razbroc merged 2 commits intomasterfrom
fix/pp-manager-request
Feb 2, 2026
Merged

fix: update PolygonPartsProcessPayload to include shouldClearEntities#85
razbroc merged 2 commits intomasterfrom
fix/pp-manager-request

Conversation

@razbroc
Copy link
Contributor

@razbroc razbroc commented Feb 2, 2026

… and adjust job processing logic

Question Answer
Bug fix ✔/✖
New feature ✔/✖
Breaking change ✔/✖
Deprecations ✔/✖
Documentation ✔/✖
Tests added ✔/✖
Chore ✔/✖

Related issues: #XXX , #XXX ...
Closes #XXX ...

Further information:

@CL-SHLOMIKONCHA
Copy link
Contributor

CL-SHLOMIKONCHA commented Feb 2, 2026

  1. run lint
  2. in swapUpdateHandler you did pass the "shouldClearEntities" boolean parameter but , in polygonPartsManagerClient you have to handle it as well and send it by the post request

Copy link
Contributor

@CL-SHLOMIKONCHA CL-SHLOMIKONCHA left a comment

Choose a reason for hiding this comment

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

see PR comments

@razbroc razbroc merged commit 762047a into master Feb 2, 2026
4 checks passed
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