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

feat: read chain id from config #308

Merged
merged 1 commit into from
May 17, 2023
Merged

Conversation

rndquu
Copy link
Member

@rndquu rndquu commented May 16, 2023

resolves #283

@netlify
Copy link

netlify bot commented May 16, 2023

Deploy Preview for ubiquibot-staging ready!

Name Link
🔨 Latest commit caf0aac
🔍 Latest deploy log https://app.netlify.com/sites/ubiquibot-staging/deploys/6463a2d9858de4000884f904
😎 Deploy Preview https://deploy-preview-308--ubiquibot-staging.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 settings.

@rndquu rndquu requested review from Steveantor and 0x4007 May 16, 2023 15:36
@@ -22,7 +22,7 @@ export const DEFAULT_BOT_DELAY = 100; // 100ms
export const DEFAULT_FOLLOWUP_TIME = "4 days"; // 4 days
export const DEFAULT_DISQUALIFY_TIME = "7 days"; // 7 days

export const DEFAULT_CHAIN_ID = 1;
export const DEFAULT_CHAIN_ID = 1; // ethereum
export const DEFAULT_RPC_ENDPOINT = "https://rpc-pay.ubq.fi/v1/mainnet";
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
export const DEFAULT_RPC_ENDPOINT = "https://rpc-pay.ubq.fi/v1/mainnet";
export const DEFAULT_RPC_ENDPOINT = "http://rpc-bot.ubq.fi/v1/mainnet";

Let's use rpc-bot.ubq.fi as a dedicated default endpoint for the bot! Also I'm pretty sure you need to use http not https.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is not part of the current issue. Pls check this PR #312

@rndquu rndquu mentioned this pull request May 16, 2023
@0x4007 0x4007 merged commit 8190a3e into ubiquity:development May 17, 2023
6 checks passed
@rndquu rndquu deleted the feat/283 branch August 15, 2023 08:13
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.

Add chainId to bot's config
2 participants