Skip to content

lazy load wallet provider on auth page#1505

Open
manav1913 wants to merge 1 commit into
gitroomhq:mainfrom
manav1913:auth-page-optimization
Open

lazy load wallet provider on auth page#1505
manav1913 wants to merge 1 commit into
gitroomhq:mainfrom
manav1913:auth-page-optimization

Conversation

@manav1913
Copy link
Copy Markdown

What kind of change does this PR introduce?

Performance optimization

Why was this change needed?

The wallet auth provider imports several Solana wallet-related dependencies that are only needed when wallet authentication is available and used.This change lazy-loads the wallet provider using next/dynamic so those dependencies are not included in the initial auth page render path, helping reduce unnecessary upfront JavaScript execution.

Other information:

eg: Did you discuss this change with anybody before working on it (not required, but can be a good idea for bigger changes). Any plans for the future, etc?
I’d also be interested in exploring additional auth page optimizations around initial bundle size and render performance in future contributions.

Checklist:

Put a "X" in the boxes below to indicate you have followed the checklist;

  • [ x] I have read the CONTRIBUTING guide.
  • [x ] I confirm I have not used AI to submit this PR or generate code for it.
  • [x ] I checked that there were no similar issues or PRs already open for this.
  • [x ] This PR fixes just ONE issue

@postiz-contribution
Copy link
Copy Markdown

postiz-contribution Bot commented May 8, 2026

Hi @manav1913! Thanks for the PR. Contributions to Postiz are gated behind an application. Please apply at https://contribute.postiz.com/p/postiz and we'll reopen this PR once you're approved.

This project also requires a signed Contributor License Agreement. You can sign it now at https://contribute.postiz.com/p/postiz/cla so you're ready as soon as your application is approved.

@postiz-contribution postiz-contribution Bot added the contribution:pending Awaiting application review label May 8, 2026
@manav1913
Copy link
Copy Markdown
Author

Submitted the contribution application form. Looking forward to contributing further.

@postiz-contribution
Copy link
Copy Markdown

Hi @manav1913! Your application for Postiz is awaiting review. We'll reopen this PR once it's approved. Status: https://contribute.postiz.com/p/postiz

This project also requires a signed Contributor License Agreement. You can sign it now at https://contribute.postiz.com/p/postiz/cla so you're ready as soon as your application is approved.

@manav1913
Copy link
Copy Markdown
Author

Hi team! The CLA signing link at https://contribute.postiz.com/p/postiz/cla is not working - getting an error when trying to access it. Could you help me sign the CLA through an alternate method so the PR can be reopened? Happy to contribute further once this is resolved. Thanks!

@egelhaus
Copy link
Copy Markdown
Collaborator

egelhaus commented May 30, 2026

Hi team! The CLA signing link at https://contribute.postiz.com/p/postiz/cla is not working - getting an error when trying to access it. Could you help me sign the CLA through an alternate method so the PR can be reopened? Happy to contribute further once this is resolved. Thanks!

Hey @manav1913 ,
according to our portal, you did indeed sign the CLA already? Did you succeed afterwards?
image

@postiz-contribution postiz-contribution Bot reopened this May 30, 2026
@postiz-contribution
Copy link
Copy Markdown

@manav1913's application for Postiz was approved. Reopening this PR.

@postiz-contribution postiz-contribution Bot added contribution:approved Approved contributor and removed contribution:pending Awaiting application review labels May 30, 2026
@postiz-agent
Copy link
Copy Markdown

postiz-agent Bot commented May 30, 2026

Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues
Licenses 0 0 0 0 0 issues
Code Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

contribution:approved Approved contributor

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants