Skip to content

Conversation

@tsotimus
Copy link

@tsotimus tsotimus commented Aug 31, 2025

Description

Current implementation:

When a request is made, a webhook notification is sent of type MEDIA_PENDING

When that request is updated/edited, no new webhook notification is sent. Not even a new MEDIA_PENDING

Desired implementation:

When a request is made, a webhook notification is sent of type MEDIA_PENDING

When that request is updated, a new webhook notification is sent of type MEDIA_PENDING_UPDATED

Added an identical PR to overseerr

Screenshot (if UI-related)

To-Dos

  • Successful build pnpm build
  • Translation keys pnpm i18n:extract
  • Database migration (if required) - No DB migration required

Issues Fixed or Closed

  • Fixes #XXXX

@tsotimus tsotimus changed the title feat(webhooks): adds a webhook for when a request is updated feat(webhooks): add MEDIA_PENDING_UPDATED webhook notification Aug 31, 2025
@tsotimus
Copy link
Author

tsotimus commented Sep 1, 2025

Having a strange discrepancy with local vs CI on the latest on this branch.
Any ideas?

Using pnpm v9.15.4 and node v22.0.0

Screenshot 2025-09-02 at 00 32 56

Copy link
Member

@gauthier-th gauthier-th left a comment

Choose a reason for hiding this comment

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

There is a mistake in the prettier configuration. You can solve it with this changes. It will be fixed in another PR.

@gauthier-th gauthier-th mentioned this pull request Sep 10, 2025
38 tasks
@github-actions github-actions bot added the merge conflict Cannot merge due to merge conflicts label Oct 30, 2025
@github-actions
Copy link

This pull request has merge conflicts. Please resolve the conflicts so the PR can be successfully reviewed and merged.

@tsotimus tsotimus requested a review from a team as a code owner November 5, 2025 20:40
@github-actions github-actions bot removed the merge conflict Cannot merge due to merge conflicts label Nov 5, 2025
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.

4 participants