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

Add new rpc to existing ones #5823

Merged
merged 1 commit into from
Sep 22, 2024
Merged

Conversation

veljkoTNFT
Copy link
Contributor

No description provided.

@veljkoTNFT
Copy link
Contributor Author

Hi @ligi if you can approve this one, we added a new RPC to our chain

Copy link
Member

@ligi ligi left a comment

Choose a reason for hiding this comment

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

please add new RPCs to the end of the list

@veljkoTNFT
Copy link
Contributor Author

veljkoTNFT commented Sep 11, 2024

I did it on purpose to make the new one first and more desirable. On my last update, that wasn't the requirement.

@ligi
Copy link
Member

ligi commented Sep 12, 2024

what last update are you referencing?

@veljkoTNFT
Copy link
Contributor Author

My last PR for updating this file. Can you pls approve, the one I added is the most desirable now to be added in the wallet?

@veljkoTNFT
Copy link
Contributor Author

If it's a must, I'll put it at the end, I just want this change to be reflected as soon as possible.

@ligi
Copy link
Member

ligi commented Sep 13, 2024

provide a link to the last PR

@veljkoTNFT
Copy link
Contributor Author

@veljkoTNFT
Copy link
Contributor Author

all ok or I need to resubmit @ligi ?

@veljkoTNFT
Copy link
Contributor Author

ping @ligi

1 similar comment
@veljkoTNFT
Copy link
Contributor Author

ping @ligi

@ligi ligi merged commit b2b1562 into ethereum-lists:master Sep 22, 2024
5 checks passed
Copy link

PR merged - please consider contributing some funds to lists.eth

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.

2 participants