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

Using json.dumps() instead of str() to ensure compatability with JSON.stringify() #825

Merged
merged 1 commit into from
Aug 24, 2023

Conversation

Dzeranov
Copy link
Contributor

@Dzeranov Dzeranov commented Aug 24, 2023

Description

Changed webhook body preparation for signing/recoving signer.

Summary of changes

Using json.dumps() instead of str() to ensure compatability with JSON.stringify()

How test the changes

docker-compose -f docker-compose.test.yml up --build test --attach test --exit-code-from test

Related issues

#824

Operational checklist

  • All new functionality is covered by tests
  • Any related documentation has been changed or added

@Dzeranov Dzeranov requested a review from flopez7 August 24, 2023 11:02
@Dzeranov Dzeranov self-assigned this Aug 24, 2023
@vercel
Copy link

vercel bot commented Aug 24, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
escrow-dashboard ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 24, 2023 11:12am
faucet-server ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 24, 2023 11:12am
fortune-exchange-oracle ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 24, 2023 11:12am
fortune-exchange-oracle-server ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 24, 2023 11:12am
fortune-job-launcher-client ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 24, 2023 11:12am
fortune-job-launcher-server ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 24, 2023 11:12am
fortune-recording-oracle ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 24, 2023 11:12am
fortune-reputation-oracle ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 24, 2023 11:12am
job-launcher-server 🛑 Canceled (Inspect) Aug 24, 2023 11:12am

@vercel vercel bot temporarily deployed to Preview – job-launcher-server August 24, 2023 11:07 Inactive
@vercel vercel bot temporarily deployed to Preview – fortune-reputation-oracle August 24, 2023 11:08 Inactive
@vercel vercel bot temporarily deployed to Preview – fortune-job-launcher-server August 24, 2023 11:09 Inactive
@vercel vercel bot temporarily deployed to Preview – fortune-job-launcher-client August 24, 2023 11:09 Inactive
@vercel vercel bot temporarily deployed to Preview – fortune-recording-oracle August 24, 2023 11:10 Inactive
@vercel vercel bot temporarily deployed to Preview – fortune-exchange-oracle August 24, 2023 11:11 Inactive
@vercel vercel bot temporarily deployed to Preview – fortune-exchange-oracle-server August 24, 2023 11:11 Inactive
@eugenvoronov
Copy link
Contributor

👍

@flopez7 flopez7 merged commit 6aac245 into develop Aug 24, 2023
12 checks passed
zhiltsov-max added a commit to zhiltsov-max/human-protocol that referenced this pull request Sep 5, 2023
@leric7 leric7 deleted the feat/cvat-change-sig-gen branch September 19, 2023 13:52
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.

3 participants