pendulum-chain / vortex

https://app.vortexfinance.co/
1 stars 1 forks source link

Sign transactions early and dump them to spreadsheet #79

Closed ebma closed 3 months ago

ebma commented 3 months ago

Notes

A new phase prepareTransactions is introduced that creates and signs all transactions at the beginning of the process.

I set up a new project 'vortex-prototype' in the Google Cloud Console and created a service account/bot that is able to access some files in our Google drive. This bot only has access to the files that are explicitly shared with it. The integration follows these guidelines.

There are two relevant files in GDrive:

I set up the relevant environment variables in the Settings of this repository already. The credentials of the service account have been shared.

Closes #58.

netlify[bot] commented 3 months ago

Deploy Preview for pendulum-pay ready!

Name Link
Latest commit 4d1414a085572d6795820de229b1cd3f3cf93c32
Latest deploy log https://app.netlify.com/sites/pendulum-pay/deploys/66b4f4c343068e00086ff60f
Deploy Preview https://deploy-preview-79--pendulum-pay.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

github-actions[bot] commented 3 months ago

yarn.lock changes

Click to toggle table visibility
| Name | Status | Previous | Current | | :- | :-: | :-: | :-: | | `@pendulum-chain/api-solang` | [UPDATED](#) | 0.4.0 | 0.6.0 |
ebma commented 3 months ago

@pendulum-chain/devs this is finally ready for review.

ebma commented 3 months ago

However, I still think it is better if we add an endpoint to the backend to push the dumpable information and then move the spreadsheet code into the backend.

True, makes sense. I'll change it.

ebma commented 3 months ago

I moved the relevant logic to the backend now but I still need to set up the environment variables for that so the deploy-preview will likely not work at the moment.

ebma commented 3 months ago

Nice catch, I removed the unused configuration again. I decided to remove the test for the google spreadsheet again because for some reason they threw CORS errors after moving into the signer service. It's not important to have them anyway but it was useful for testing during the implementation.

ebma commented 3 months ago

The setup of the environment variables of the backend will happen as part of https://github.com/pendulum-chain/tasks/issues/381.