Skip to content

refactor: remove redux from validator page#634

Merged
ckniffen merged 1 commit intostagingfrom
refactor/validators-redux
Mar 21, 2023
Merged

refactor: remove redux from validator page#634
ckniffen merged 1 commit intostagingfrom
refactor/validators-redux

Conversation

@ckniffen
Copy link
Copy Markdown
Collaborator

High Level Overview of Change

Leverage useWindowSize instead.

Type of Change

  • Refactor (non-breaking change that only restructures code)

Leverage `useWindowSize` instead.
@ckniffen ckniffen requested review from JST5000 and jonathanlei March 20, 2023 18:33
Copy link
Copy Markdown
Contributor

@JST5000 JST5000 left a comment

Choose a reason for hiding this comment

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

LGTM

@ckniffen ckniffen merged commit 2c5d020 into staging Mar 21, 2023
@ckniffen ckniffen deleted the refactor/validators-redux branch March 21, 2023 12:53
@mvadari mvadari mentioned this pull request May 20, 2024
1 task
achowdhry-ripple added a commit that referenced this pull request Apr 8, 2025
## High Level Overview of Change

This PR removes the last of redux from the codebase. The only two pieces
left were Account and Token.

It also removes all mentions of gravatars.

This also reduces the size of the build by about 35kb.

There were some build issues that required regenerating the
package-lock, which is almost all of the diff.

### Context of Change

* #649
* #640
* #635
* #634
* #1100

### Type of Change

- [x] Refactor (non-breaking change that only restructures code)

### TypeScript/Hooks Update

N/A

## Test Plan

Tests were fixed to handle the refactor.

---------

Co-authored-by: Ashray Chowdhry <achowdhry@ripple.com>
heap-coder added a commit to heap-coder/explorer that referenced this pull request Sep 27, 2025
## High Level Overview of Change

This PR removes the last of redux from the codebase. The only two pieces
left were Account and Token.

It also removes all mentions of gravatars.

This also reduces the size of the build by about 35kb.

There were some build issues that required regenerating the
package-lock, which is almost all of the diff.

### Context of Change

* ripple/explorer#649
* ripple/explorer#640
* ripple/explorer#635
* ripple/explorer#634
* ripple/explorer#1100

### Type of Change

- [x] Refactor (non-breaking change that only restructures code)

### TypeScript/Hooks Update

N/A

## Test Plan

Tests were fixed to handle the refactor.

---------

Co-authored-by: Ashray Chowdhry <achowdhry@ripple.com>
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