Skip to content

[@astrojs/prefetch]: Prevent prefetching current page#5009

Merged
matthewp merged 5 commits intowithastro:mainfrom
panwauu:pretech-not-same-page
Oct 7, 2022
Merged

[@astrojs/prefetch]: Prevent prefetching current page#5009
matthewp merged 5 commits intowithastro:mainfrom
panwauu:pretech-not-same-page

Conversation

@panwauu
Copy link
Copy Markdown
Contributor

@panwauu panwauu commented Oct 7, 2022

Changes

Testing

  • Added prefetch link to "/" to the test page
  • Added tests to check that the path "/" is only requested once
  • Refactored the unnecessary use of await keyword

Docs

As this is a bugfix, no docs change is needed.

Fixes #4963

@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Oct 7, 2022

🦋 Changeset detected

Latest commit: 23c4903

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@github-actions github-actions bot added the pkg: integration Related to any renderer integration (scope) label Oct 7, 2022
@panwauu
Copy link
Copy Markdown
Contributor Author

panwauu commented Oct 7, 2022

Failed test seems unrelated

@matthewp matthewp merged commit 92b27e9 into withastro:main Oct 7, 2022
@astrobot-houston astrobot-houston mentioned this pull request Oct 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pkg: integration Related to any renderer integration (scope)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[@astrojs/prefetch] Prevent prefetching current page

2 participants