Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Add Preconnect if the server url isn't the same as backend url #10236

Open
FelixMalfait opened this issue Feb 16, 2025 · 2 comments · May be fixed by #10554
Open

Add Preconnect if the server url isn't the same as backend url #10236

FelixMalfait opened this issue Feb 16, 2025 · 2 comments · May be fixed by #10554
Labels
for experienced contributor type: chore

Comments

@FelixMalfait
Copy link
Member

We should add a preconnect to the server url, this should help improve the loading time a bit

Image
@SujithThirumalaisamy
Copy link
Contributor

Have a small note on this. Please point me on the mistakes here. Happy to work on this aswell.

  • Add page-prefetch util in the ui/utilities
  • Add utility function to fetch the necessary urls for each page from a config.

Came to this idea in reference to how we have implemented page-title. Will be happy to here remarks on this and start to work on this

This was referenced Feb 27, 2025
Closed
Open
@RajvirS99
Copy link

@FelixMalfait I am really sorry, for the previous linking of the PR. When I pushed the code, I checked the branch was a lot older than the latest main branch so, I fixed things and made another PR and linked to this issue.

This is the correct PR - #10554

Kindly let me know, There is any change I need to make to make sure I have made the contribution according to the coding style of the application. Otherwise, I feel this seems to fix the issue by adding the preconnect in the head section!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
for experienced contributor type: chore
Projects
🎯 Roadmap & Sprints
Status: No status
Milestone
No milestone
3 participants
@FelixMalfait @RajvirS99 @SujithThirumalaisamy