-
Notifications
You must be signed in to change notification settings - Fork 236
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
Add a flag query to Netlify Identity links #157
Comments
who is "we" here? i dont know who would own this |
Netlify platform. We'd need to update various identity links, such as those in verification emails, to use the flag query, and supporting changes would also potentially occur in this repo. Again, raising here instead of internally because it's a community issue. Sent with GitHawk |
Any updates on this or anything I can contribute to? Would really want to solve this somehow so all the users don't have to load |
Merged your Gatsby PR as a stop gap 👍 |
That would 100% make this request unnecessary 👍 |
This technically isn't an ask for this project, but I'm adding it here for community visibility as it's related to gatsbyjs/gatsby#9565.
Loading the identity widget on every page, especially in an SSG with auto code splitting like Gatsby, often adds unnecessary weight. We should consider adding some sort of flag query to Netlify Identity links so that the widget can be dynamically imported only when necessary. Something like
netlify_identity_redirect=true
rather than justconfirmation_token=xxx
, which could technically be from anywhere.Can we determine if this is feasible or if an alternate approach should be recommended? The PR linked above is an example of what I'd like to help folks avoid, copying internal logic from this repo to determine if the url requires the widget.
The text was updated successfully, but these errors were encountered: