Skip to content

refactor: retrieve country data on page load#643

Merged
ckniffen merged 1 commit intorefactor/networkfrom
refactor/dynamic-countries-data
Mar 28, 2023
Merged

refactor: retrieve country data on page load#643
ckniffen merged 1 commit intorefactor/networkfrom
refactor/dynamic-countries-data

Conversation

@ckniffen
Copy link
Copy Markdown
Collaborator

High Level Overview of Change

Moves country data out of the js bundle and instead load it when the user visits the node page. This reduces the bundle by 39kb (gzipped).

Type of Change

  • Refactor (non-breaking change that only restructures code)
  • Tests (You added tests for code that already exists, or your new feature included in this PR)

Moves country data out of the js bundle and instead load it when the
user visits the node page. This reduces the bundle by 39kb (gzipped).
@ckniffen ckniffen requested review from JST5000 and pdp2121 March 24, 2023 19:13
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 c5c6a14 into refactor/network Mar 28, 2023
@ckniffen ckniffen deleted the refactor/dynamic-countries-data branch March 28, 2023 00:08
@ckniffen ckniffen restored the refactor/dynamic-countries-data branch March 28, 2023 01:27
ckniffen added a commit that referenced this pull request Mar 28, 2023
Moves country data out of the js bundle and instead load it when the
user visits the node page. This reduces the bundle by 39kb (gzipped).

Merged #643 too early.  Needed #640.
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