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 Suiware Faucet Chrome Extension #35

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

kkomelin
Copy link
Contributor

No description provided.

@yanganto
Copy link
Contributor

yanganto commented Dec 26, 2024

Hi @kkomelin
It is awesome to see a new extension here. I try the extension and find out there is bug in the extension, and the issue page of the repo is closed. Would you open the issue tag and also solve these bugs.

  1. the extension does not memory previous devnet and testnet setting.
  2. the testnet is not really work with following error from my end, and I only try once.
    error

@kkomelin
Copy link
Contributor Author

kkomelin commented Dec 26, 2024

Thank you very much @yanganto for your feedback!

Let me address the issues you mentioned one by one...

  1. I've opened the issue queue for the extension repository. Good catch! https://github.com/suiware/suiware-faucet-extension/issues

  2. I've created an issue for your feature request Preserve chosen network suiware/suiware-faucet-extension#1

  3. The bug you described (Testnet faucet doesn't work suiware/suiware-faucet-extension#2) unfortunately cannot be fixed currently because of the limitations of the official faucet API which the extension uses. However, there is some hope that the official faucet API limitations will be revised in the future. I reported the issue to the team.

Additionally, I've renamed the repository from suiware/suiware-faucet to suiware/suiware-faucet-extension to reflect the purpose better. Also, I've added current limitations to the Readme, highlighting the fact that the extension is experimental and it has issues.

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