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.

Sign up for GitHub

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

Jump to bottom

Script executor temporal worker & similar identities merger workflows #2391

Merged
merged 21 commits into from May 13, 2024
Merged

Script executor temporal worker & similar identities merger workflows #2391

merged 21 commits into from May 13, 2024

Conversation

epipav
Copy link
Collaborator

@epipav epipav commented May 6, 2024 •
edited

Changes proposed ✍️

What

copilot:summary
​
copilot:poem

Why

How

copilot:walkthrough

Checklist ✅

  • Label appropriately with Feature, Improvement, or Bug.
  • Add screenshots to the PR description for relevant FE changes
  • New backend functionality has been unit-tested.
  • API documentation has been updated (if necessary) (see docs on API documentation).
  • Quality standards are met.

@epipav epipav self-assigned this May 6, 2024
@epipav epipav added the Feature Created by Linear-GitHub Sync label May 6, 2024
@epipav epipav marked this pull request as ready for review May 6, 2024 05:59
@epipav epipav requested a review from themarolt May 6, 2024 06:05
epipav added 17 commits May 6, 2024 08:45
@epipav
first tests passing
1e3d597
@epipav
testing fix for fetch error `Cannot construct a Request with a Reques…
b4ad8d7 
…t object that has already been used`
@epipav
some logging for debugging
6a03680
@epipav
using axios for merging endpoint call
9c212d7
@epipav
switched to axios insteaed of fetch api, remove excessive logging
4ef3f7d
@epipav
remove merge result logging
5d74659
@epipav
increase page size
35ecca4
@epipav
handle member not found gracefully when merging
c9be4d9
@epipav
handle 404s gracefully
f4bf75a
@epipav
logging errors for better handling
b163710
@epipav
error handling on merge
95e3280
@epipav
change backoff coefficient for merge activity
4a4b3b9
@epipav
better retries for merging
a241802
@epipav
preent throwing errors when merging members, instead skip current mer…
cdb7f22 
…ge action
@epipav
exclude problematic member from merges
9b42ea0
@epipav
exclude corrupt members
7ad5152
@epipav
Merge branch 'crowd-linux' into feature/script-executor-temporal-worker
1b599db
@epipav epipav merged commit f0a9160 into crowd-linux May 13, 2024
6 checks passed
@epipav epipav deleted the feature/script-executor-temporal-worker branch May 13, 2024 08:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@themarolt themarolt themarolt approved these changes

Assignees

@epipav epipav

Labels
Feature Created by Linear-GitHub Sync
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@epipav @themarolt