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

Create pool - functionality #2545

Merged
merged 17 commits into from
Dec 4, 2024
Merged

Conversation

kattylucy
Copy link
Collaborator

@kattylucy kattylucy commented Nov 26, 2024

Creation of pool redesign

#2529

Approvals

  • Dev

@kattylucy kattylucy changed the base branch from main to create_pool_structure November 26, 2024 09:27
Copy link

github-actions bot commented Nov 26, 2024

PR deployed in Google Cloud
URL: https://app-pr2545.k-f.dev
Commit #: b0f5982
To access the functions directly check the corresponding deploy Action

Copy link

github-actions bot commented Nov 26, 2024

PR deployed in Google Cloud
URL: https://pr2545-app-ff-production.k-f.dev
Commit #: b0f5982
To access the functions directly check the corresponding deploy Action

Base automatically changed from create_pool_structure to create_pool_redesign November 27, 2024 15:29
@kattylucy kattylucy force-pushed the create_pool_functionality branch from 41696cc to d5c9bd3 Compare November 27, 2024 15:32
@kattylucy kattylucy force-pushed the create_pool_functionality branch from d5c9bd3 to 36cd71c Compare November 27, 2024 15:38
@kattylucy kattylucy force-pushed the create_pool_functionality branch from 121e56e to 945bf45 Compare November 28, 2024 09:49
@kattylucy kattylucy marked this pull request as ready for review December 3, 2024 12:20
@@ -40,20 +40,28 @@ export const IssuerCategoriesSection = () => {
<>
{form.values.issuerCategories.map((_, index) => (
Copy link
Collaborator

Choose a reason for hiding this comment

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

now that it's used, better to give _ a name

@kattylucy kattylucy force-pushed the create_pool_functionality branch from 9e9f599 to b0f5982 Compare December 3, 2024 15:50
label="Reviewer avatar"
placeholder="Choose file"
accept="image/*"
Copy link
Collaborator

Choose a reason for hiding this comment

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

I would limit the file types even more, e/g accept="image/png, image/jpeg, image/jpg"

@kattylucy kattylucy merged commit d912f29 into create_pool_redesign Dec 4, 2024
12 checks passed
@kattylucy kattylucy deleted the create_pool_functionality branch December 4, 2024 11:04
kattylucy added a commit that referenced this pull request Dec 13, 2024
* Fix ts error and change logic for onboarding values

* Add create pool existing functionality

* Cleanup types

* cleanup

* Add deposit banner

* Fix linter errors

* Add metadata values

* Cleanup types

* Add onboarding functionality and UI fixes

* Add proxies functionality

* Fix ts errors

* Add create pool dialog

* Add dialogs

* Add review feedback

* wip

* Add waiting before redirecting to avoid error

* Remove default empty pool fee
kattylucy added a commit that referenced this pull request Dec 17, 2024
* Fix ts error and change logic for onboarding values

* Add create pool existing functionality

* Cleanup types

* cleanup

* Add deposit banner

* Fix linter errors

* Add metadata values

* Cleanup types

* Add onboarding functionality and UI fixes

* Add proxies functionality

* Fix ts errors

* Add create pool dialog

* Add dialogs

* Add review feedback

* wip

* Add waiting before redirecting to avoid error

* Remove default empty pool fee
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.

3 participants