You can not select more than 25 topics Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.

Jakub Serafin a9f4cfde90 (core) API reworked to use POST to create webhook and DELET to remove it ... Summary: introduces POST /api/docs/{docId}/webhooks and DELETE /api/docs/{docId}/webhooks/{webhookId} on place of old _subscribe and _unsubscribe endpoints. Remove checking for unsubscribeKey while deleting webhook - only owner can delete webhook using DELETE endpoint. subscription key is still needed for _unsubscribe endpoint. old _unsubscribe and _subscribe endpoints are still active and work as before - no changes there. Posting schema: ``` POST /api/docs/[docId]/webhooks ``` Request Body: ``` { "webhooks": [ { "fields": { "url": "https://webhook.site/3bd02246-f122-445e-ba7f-bf5ea5bb6eb1", "eventTypes": [ "add", "update" ], "enabled": true, "name": "WebhookName", "memo": "just a text", "tableId": "Table1" } }, { "fields": { "url": "https://webhook.site/3bd02246-f122-445e-ba7f-bf5ea5bb6eb2", "eventTypes": [ "add", ], "enabled": true, "name": "OtherWebhookName", "memo": "just a text", "tableId": "Table1" } } ] } ``` Expected response: WebhookId for each webhook posted: ``` { "webhooks": [ { "id": "85c77108-f1e1-4217-a50d-acd1c5996da2" }, { "id": "d87a6402-cfd7-4822-878c-657308fcc8c3" } ] } ``` Deleting webhooks: ``` DELETE api/docs/[docId]/webhooks/[webhookId] ``` there is no payload in DELETE request. Therefore only one webhook can be deleted at once Response: ``` { "success": true } ``` Test Plan: Old unit test improved to handle new endpoints, and one more added to check if endpoints are in fact created/removed Reviewers: alexmojaki Reviewed By: alexmojaki Subscribers: paulfitz, alexmojaki Differential Revision: https://phab.getgrist.com/D3916		12 months ago
..
client	(core) bump mocha version to allow parallel tests; move more tests to core	1 year ago
client-harness	(core) bump mocha version to allow parallel tests; move more tests to core	1 year ago
common	(core) Add Support Grist page and nudge	12 months ago
deployment	Adding new lines at the end	2 years ago
fixtures	skip building test harness in docker image (#551)	1 year ago
formula-dataset	(core) Billing for formula assistant	12 months ago
gen-server	(core) Persist forks in home db	1 year ago
nbrowser	(core) When a page starts with a number, don't treat it as an emoji	12 months ago
server	(core) API reworked to use POST to create webhook and DELET to remove it	12 months ago
timings	run grist-core test batches in parallel (#444)	1 year ago
.eslintrc.js	Fixing all eslint's reported error	1 year ago
chai-as-promised.js	(core) Faster builds all around.	2 years ago
declarations.d.ts	(core) bump mocha version to allow parallel tests; move more tests to core	1 year ago
init-mocha-webdriver.js	(core) bump mocha version to allow parallel tests; move more tests to core	1 year ago
mocha.opts	run grist-core test batches in parallel (#444)	1 year ago
report-why-tests-hang.js	(core) bump mocha version to allow parallel tests; move more tests to core	1 year ago
setupPaths.js	(core) bump mocha version to allow parallel tests; move more tests to core	1 year ago
split-tests.js	(core) bump mocha version to allow parallel tests; move more tests to core	1 year ago
testUtils.ts	(core) add missing tsconfig file that affects IDEs	2 years ago
test_under_docker.sh	Fix GREP_TESTS unbound error in test_under_docker.sh (#549)	1 year ago
tsconfig.json	(core) Faster builds all around.	2 years ago
upgradeDocument	add test/upgradeDocument utility	1 year ago
upgradeDocumentImpl.ts	add test/upgradeDocument utility	1 year ago
utils.js	(core) bump mocha version to allow parallel tests; move more tests to core	1 year ago
xunit-file.js	(core) bump mocha version to allow parallel tests; move more tests to core	1 year ago