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

added a function to notify the status of deployment to backend #2

Merged
merged 1 commit into from
Jan 24, 2025

Conversation

Arjxm
Copy link
Collaborator

@Arjxm Arjxm commented Jan 16, 2025

This pull request includes several changes to the index.js and network.js files to enhance the deployment notification process and improve code consistency by standardizing formatting. The most important changes include adding a new function to send deployment notifications and updating the RPC URL configurations.

Enhancements to deployment notifications:

  • Added sendNotificationToBackend function to send deployment notifications to the backend service. This function constructs a payload with relevant deployment data and sends it via a POST request. (index.js)
  • Updated the executeDeploy function to send deployment notifications at the start, success, and failure points of the deployment process. This ensures that the backend service is informed about the deployment status at each critical stage. (index.js)

Code formatting improvements:

  • Standardized the formatting of the RPC_URL object by removing unnecessary newlines and aligning the URL arrays. (network.js) [1] [2]
  • Updated the getLatestBlockNumber function to use consistent indentation and quotation marks, improving code readability. (network.js) [1] [2]

@Arjxm Arjxm merged commit 0b43ac3 into main Jan 24, 2025
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.

1 participant