Closed trajan0x closed 2 weeks ago
[!CAUTION]
Review failed
The pull request is closed.
This pull request introduces significant enhancements to the Synapse REST API documentation and functionality. A new JavaScript file, generate-swagger.js
, is added to generate OpenAPI documentation using the swagger-jsdoc
library. The package.json
is updated to include a new script for generating Swagger documentation and a new dependency. Additionally, modifications are made to the bridgeController
and bridgeRoute
files to incorporate transaction data encoding and update the response schema, respectively. A new configuration file for Swagger is also created, and the existing swagger.json
is extensively revised.
File Path | Change Summary |
---|---|
packages/rest-api/generate-swagger.js |
New file created to generate Swagger documentation using swagger-jsdoc . |
packages/rest-api/package.json |
New script generate-swagger added; dependency @ethersproject/abi introduced; minor formatting change. |
packages/rest-api/src/controllers/bridgeController.ts |
New constant BRIDGE_INTERFACE added; transaction data encoding implemented in the bridgeController function. |
packages/rest-api/src/routes/bridgeRoute.ts |
Response schema updated to include new property txData: string for the /bridge endpoint. |
packages/rest-api/swagger.config.js |
New configuration file created for Swagger API documentation. |
packages/rest-api/swagger.json |
Extensive updates to API metadata, parameters, response schemas, and overall structure. |
generate-swagger.js
.generate-swagger.js
functionality.swagger.json
to openapi.json
, aligning with the focus on generating OpenAPI documentation.size/m
, M-docs
In the land of code where rabbits play,
New docs emerge to light the way.
With swagger's might, our paths align,
Encoding bridges, oh how they shine!
Hop along, let’s celebrate,
For clearer APIs, we can’t wait! 🐇✨
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Latest commit: |
a1ebff3
|
Status: | ✅ Deploy successful! |
Preview URL: | https://f1dfc000.sanguine-fe.pages.dev |
Branch Preview URL: | https://fix-api-fix.sanguine-fe.pages.dev |
Description A clear and concise description of the features you're adding in this pull request.
Additional context Add any other context about the problem you're solving.
Metadata
Summary by CodeRabbit
New Features
/bridge
endpoint response to include encoded transaction data (txData
).Updates
/bridge
endpoint for improved functionality.Bug Fixes