Skip to content

Commit

Permalink
fix: fix bugs found while testing
Browse files Browse the repository at this point in the history
  • Loading branch information
JoeCap08055 committed Sep 23, 2024
1 parent 2db9cdc commit 0840d1a
Show file tree
Hide file tree
Showing 6 changed files with 7 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -88,7 +88,7 @@ export function createWebhookRsp(
}

if (transactionType === TransactionType.SIWF_SIGNUP && isSiwfOpts(options)) {
return response as SIWFWebhookRsp;
return { ...response, ...options } as SIWFWebhookRsp;
}

throw new Error(`Invalid transaction type ${transactionType} for webhook response`);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -170,6 +170,7 @@ export class TxnNotifierService
accountId: siwfTxnValues.address,
handle: siwfTxnValues.handle,
});
webhookResponse = response;

this.logger.log(
`SIWF: ${siwfTxnValues.address} Signed up handle ${response.handle} for msaId ${response.msaId} delegated to provider ${siwfTxnValues.newProvider}.`,
Expand Down
2 changes: 2 additions & 0 deletions docker-compose.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -44,6 +44,8 @@ x-account-service-env: &account-service-env
TRUST_UNFINALIZED_BLOCKS: true
WEBHOOK_BASE_URL: 'http://mock-webhook-server:${ACCOUNT_WEBHOOK_PORT:-3001}/webhooks/account-service'
GRAPH_ENVIRONMENT_TYPE: Mainnet
CACHE_KEY_PREFIX: 'account:'
DEBUG: true

services:
redis:
Expand Down
1 change: 0 additions & 1 deletion libs/types/src/enums/announcement-type.enum.ts
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,6 @@ export enum AnnouncementType {
Reaction = 4,
Profile = 5,
Update = 6,
PublicFollows = 113,
}

// eslint-disable-next-line no-shadow
Expand Down
4 changes: 2 additions & 2 deletions openapi-specs/content-announcement.openapi.json
Original file line number Diff line number Diff line change
Expand Up @@ -33,8 +33,8 @@
"components": {
"schemas": {
"AnnouncementType": {
"enum": [0, 2, 3, 4, 5, 6, 113],
"x-enum-varnames": ["Tombstone", "Broadcast", "Reply", "Reaction", "Profile", "Update", "PublicFollows"]
"enum": [0, 2, 3, 4, 5, 6],
"x-enum-varnames": ["Tombstone", "Broadcast", "Reply", "Reaction", "Profile", "Update"]
},
"AnnouncementResponse": {
"type": "object",
Expand Down
1 change: 1 addition & 0 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -82,6 +82,7 @@
"test:e2e:content-publishing": "dotenvx run -f env-files/content-publishing.template.env -- jest --testRegex \".e2e-spec.ts\" --detectOpenHandles 'content-publishing*'",
"test:e2e:content-watcher": "dotenvx run -f env-files/content-watcher.template.env -- jest --testRegex \".e2e-spec.ts\" --detectOpenHandles 'content-watcher*'",
"test:e2e:graph": "dotenvx run -f env-files/graph.template.env -- jest --testRegex \".e2e-spec.ts\" --detectOpenHandles 'graph*'",
"test:k6:account": "for t in apps/account-api/k6-test/*.k6.js ; do k6 run ${t} ; done",
"format": "prettier --write .",
"lint": "eslint apps/**/*.[tj]s libs/**/*.[tj]s",
"lint:account": "eslint apps/account*/**/*.[tj]s libs/account-lib/**/*.[tj]s",
Expand Down

0 comments on commit 0840d1a

Please sign in to comment.