Skip to content

Extract FeeRateSelector shared component - #539

Merged
RodriFS merged 2 commits into
mainfrom
move-fee-selection-to-common-component
Jul 31, 2026
Merged

Extract FeeRateSelector shared component#539
RodriFS merged 2 commits into
mainfrom
move-fee-selection-to-common-component

Conversation

@RodriFS

@RodriFS RodriFS commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

@RodriFS
RodriFS force-pushed the move-fee-selection-to-common-component branch from 7b7731f to d410166 Compare July 24, 2026 12:38
@RodriFS
RodriFS force-pushed the move-fee-selection-to-common-component branch from d410166 to 9c87675 Compare July 24, 2026 12:42
@RodriFS
RodriFS force-pushed the add-coin-selection-to-transfer-modal branch from 6870f71 to 45131ea Compare July 24, 2026 13:01
@RodriFS
RodriFS force-pushed the move-fee-selection-to-common-component branch from 9c87675 to 350de4b Compare July 24, 2026 13:01
Base automatically changed from add-coin-selection-to-transfer-modal to main July 27, 2026 07:08
manumonti
manumonti previously approved these changes Jul 27, 2026

@manumonti manumonti left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM! I had a bit of a hard time reviewing because of the code formatting though

@RodriFS
RodriFS dismissed manumonti’s stale review July 28, 2026 09:48

The merge-base changed after approval.

@RodriFS
RodriFS force-pushed the move-fee-selection-to-common-component branch from 350de4b to 494a1b7 Compare July 28, 2026 09:48
@RodriFS

RodriFS commented Jul 28, 2026

Copy link
Copy Markdown
Contributor Author

LGTM! I had a bit of a hard time reviewing because of the code formatting though

that's because it had conflicts when you reviewed it and it showed also changes from main 😅. you can take another look now

@markettes markettes left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@RodriFS
RodriFS enabled auto-merge (squash) July 31, 2026 08:28
@RodriFS
RodriFS merged commit ae866d7 into main Jul 31, 2026
5 checks passed
@RodriFS
RodriFS deleted the move-fee-selection-to-common-component branch July 31, 2026 08:29
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