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

Prevent SmartSuite webhooks from expiring #14384

Closed
malexanderlim opened this issue Oct 23, 2024 · 4 comments · Fixed by #14386
Closed

Prevent SmartSuite webhooks from expiring #14384

malexanderlim opened this issue Oct 23, 2024 · 4 comments · Fixed by #14386
Assignees

Comments

@malexanderlim
Copy link
Collaborator

malexanderlim commented Oct 23, 2024

https://developers.smartsuite.com/docs/solution-data/webhooks/webhooks-overview#webhook-expiration

Webhook Expiration
You must call list payloads at least once every 7 days for the webhook to stay active. After 7 days of inactivity we will disable notifications for your webhook. Webhooks in this state for a further 7 days will be deleted, along with their payloads. Calling list payloads during the 7 day inactive period will re-enable the webhook and extend the timeout period an additional 7 days.

@malexanderlim malexanderlim converted this from a draft issue Oct 23, 2024
@malexanderlim malexanderlim changed the title Add List Payloads call to SmartSuite webhooks to keep from expiring Prevent SmartSuite webhooks from expiring Oct 23, 2024
@malexanderlim malexanderlim moved this from To Do to Prioritized in Component (Source and Action) Backlog Oct 23, 2024
@michelle0927 michelle0927 moved this from Prioritized to Doing in Component (Source and Action) Backlog Oct 23, 2024
@michelle0927 michelle0927 moved this from Doing to Ready for PR Review in Component (Source and Action) Backlog Oct 23, 2024
@GTFalcao GTFalcao moved this from Ready for PR Review to Ready for QA in Component (Source and Action) Backlog Oct 23, 2024
@vunguyenhung vunguyenhung moved this from Ready for QA to Changes Required in Component (Source and Action) Backlog Oct 24, 2024
@vunguyenhung vunguyenhung moved this from Changes Required to Ready for QA in Component (Source and Action) Backlog Oct 24, 2024
@vunguyenhung vunguyenhung moved this from Ready for QA to Changes Required in Component (Source and Action) Backlog Oct 24, 2024
@vunguyenhung
Copy link
Collaborator

Hello everyone, I have tested this PR and there're some test cases failed or needed improvement.

Please check the test report below for more information
https://vunguyenhung.notion.site/Prevent-SmartSuite-webhooks-from-expiring-128bf548bb5e81d4ac20d77206c5d3fa

@michelle0927
Copy link
Collaborator

Hi @vunguyenhung , I think the reason the events aren't being emitted is that the "Solution" is set to Status Reports, but the records are being added to the Project Management solution.

Screenshot 2024-10-24 at 11 31 48 AM

@vunguyenhung
Copy link
Collaborator

Oh my bad @michelle0927, will test it again

@vunguyenhung vunguyenhung moved this from Changes Required to Ready for QA in Component (Source and Action) Backlog Oct 24, 2024
@vunguyenhung vunguyenhung moved this from Ready for QA to Ready for Release in Component (Source and Action) Backlog Oct 25, 2024
@vunguyenhung
Copy link
Collaborator

Hi everyone, all test cases are passed! Ready for release!

Test report
https://vunguyenhung.notion.site/Prevent-SmartSuite-webhooks-from-expiring-128bf548bb5e81d4ac20d77206c5d3fa

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging a pull request may close this issue.

3 participants