-
Notifications
You must be signed in to change notification settings - Fork 35
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
Fix docker build #451
Closed
Closed
Fix docker build #451
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…ency-testnet-faucet into frequency_init_changes
frequency modifications
testing new backend url
Because it was bugging me. Closes #9 Co-authored-by: Wes Biggs <[email protected]>
Merging the upstream changes <img width="1029" alt="image" src="https://github.com/LibertyDSNP/frequency-testnet-faucet/assets/1252199/e29ac14e-8895-4fe2-bba3-8c6205f487fb"> --------- Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: Javier Bullrich <[email protected]> Co-authored-by: Przemek Rzad <[email protected]> Co-authored-by: Yuri Volkov <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Pierre Besson <[email protected]> Co-authored-by: Leonardo Custodio <[email protected]> Co-authored-by: Alexander Samusev <[email protected]> Co-authored-by: Maksym H <[email protected]> Co-authored-by: BulatSaif <[email protected]> Co-authored-by: Javier Bullrich <[email protected]>
Simple switch back to sqlite before we deploy the backend.
## Goal The goal of this PR is to fix the missing Rococo from the dropdown and correctly default to Frequency in the UI ## Checklist - [x] PR Self-Review and Commenting ## How To Test the Changes 1. Clone the pr branch 2. `cd client` 3. `yarn dev` 4. See the dropdowns correctly <img width="570" alt="image" src="https://github.com/frequency-chain/testnet-faucet/assets/1252199/4948a657-c60f-4627-af42-9584486459a8">
## Goal The goal of this PR is to publish docker images to GitHub ## Checklist - [x] PR Self-Review and Commenting - [x] Docs updated - [x] Tests added ## How To Test the Changes - Test Run: https://github.com/frequency-chain/testnet-faucet/actions/runs/8527012881 - Package Details: https://github.com/frequency-chain/testnet-faucet/pkgs/container/testnet-faucet --------- Co-authored-by: Dmitri <[email protected]>
## Goal The goal of this PR is to enable to Frequency Paseo faucet ## Discussion This alters more than I would like, but I wanted the dropdown to work ## Checklist - [x] PR Self-Review and Commenting ## Screenshots ![image](https://github.com/frequency-chain/testnet-faucet/assets/1252199/2da5e8d1-4842-4997-8c6d-ce27d665ba2d)
## Problem Needed a bit better errors, but really, needed to get the Paseo faucet working again. ## Solution * update all `@polkadot*` packages * update any other needed packages because of that * make any necessary changes as a result * validate the address before calling fetch. If it's invalid, make a nice message. * Be smarter about detecting some errors. * Don't keep a .env file around
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Docker build was broken because the top level
yarn run build
was broken.