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

Test new #1773

Closed
wants to merge 17 commits into from
Closed

Test new #1773

wants to merge 17 commits into from

Conversation

mishramonalisha76
Copy link
Collaborator

Pull Request Template

Ticket Number

Description

  • Problem/Feature:

Type of Change

  • Bug fix
  • New feature
  • Code refactor
  • Documentation update
  • Other (please describe):

Checklist

  • Quick PR: Is this a quick PR? Can be approved before finishing a coffee.
    • Quick PR label added
  • Not Merge Ready: Is this PR dependent on some other PR/tasks and not ready to be merged right now.
    • DO NOT Merge PR label added

Frontend Guidelines

Build & Testing

  • No errors in the build terminal
  • Engineer has tested the changes on their local environment
  • Engineer has tested the changes on deploy preview

Screenshots/Video with Explanation

  • Before: Explain the previous behavior

  • After: What's changed now

Additional Context

Review & Approvals

  • Self-review completed
  • Code review by at least one other engineer
  • Documentation updates if applicable

Notes

abhishek-01k and others added 17 commits July 8, 2024 13:25
…push-dapp into Feat-New-Channel-Creation-Flow
…push-dapp into Feat-New-Channel-Creation-Flow
* added basic layout

* finished ui

* added fixes

* addes stepper

* added layout of select

* fixed width of the popover

* fixed alignment

* added review comments

* fixed form code

* removed Box from select

* added form context wrapper

* moved validation outside

* Update index.ts
* New Channel Creation Flow based on new UI

* Function implementation of the create Channel

* Added New Page when chain is diff

* Fixed the navigation for channel creation flow

* Pulled the main to change the theme way

* Removed the new createChannel route

* Removed faucet duplication

* Fixed issue with the text and text color

* Fixed the imports and other issues

* Resolved the stepper issue

* Fixed the input validation and also added validaiton msg

* fxed error validations

* import fixes

* refactoring done

* Fixed the stepper steps prop

* File upload moved to blocks and form is also corrected for ChannelInfo

* Created Alert Block just for error

* Removed hidden prop fro file upload and also fixed the stake fees error comment

---------

Co-authored-by: rohitmalhotra1420 <[email protected]>
…push-dapp into Feat-New-Channel-Creation-Flow
* added layout

* send notif

* added formik

* fixed mobile view

* fixed mobile view

* fixed enter key

* added missing things in select

* channel dashboard fix

* fixed the navigation issue

* fixed the navigation issue

* fixed review issues
* Channel Dashboard UI

* Added new Dashboard UI

* Gave new route to Edit Channel

* Fixed all the imports and minor changes

* Removed ChainDetails config file

* Added Navigation to Add new chain in dashboard

* Merged New Channel Creation flow and new dashboard

* Fixed the tooltip and edit channel

* Added retry logic for channel dashboard fetching

* Fixed the editchannel and tooltip minor UI bugs

* Fixed the inline error messages

* Fixed the issue regarding theme

* Fixed the comments regarding Delegate, subgraph, footer

* Create Channel Flow and Channel Dashboard is merged

* fixed a bug

* Resolved all the issues

* Fixed the size of Logo component in the MenuItem

* Fixed the Menu Item Icons and text color

* Fixed the Menu Items in the channeldashboardheader

* Added color prop to the MenuItem

* Fixed the MenuItem and added listItem semantic

* Added light and dark mode to listItem semantic

---------

Co-authored-by: Monalisha Mishra <[email protected]>
Copy link

PR Preview Action v1.4.7
🚀 Deployed preview to https://push-protocol.github.io/push-dapp/pr-preview/pr-1773/
on branch gh-pages at 2024-07-29 10:26 UTC

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