Skip to content

Conversation

katzino
Copy link
Contributor

@katzino katzino commented Oct 15, 2025

supports https://github.com/apify/apify-web/issues/5377

  • turned out that location from docusaurus doesn't provide url 😂
  • using window.location instead does the trick

@katzino katzino added this to the 125th sprint - Web team milestone Oct 15, 2025
@katzino katzino self-assigned this Oct 15, 2025
@katzino katzino added bug Something isn't working. t-web Issues with this label are in the ownership of the web team. labels Oct 15, 2025
cursor[bot]

This comment was marked as outdated.

@apify-service-account
Copy link

Preview for this PR was built for commit 3959a4c and is ready at https://pr-2017.preview.docs.apify.com!

@apify-service-account
Copy link

Preview for this PR was built for commit 393f217 and is ready at https://pr-2017.preview.docs.apify.com!

@katzino
Copy link
Contributor Author

katzino commented Oct 15, 2025

@cursor review

Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Bugbot reviewed your changes and found no bugs!


Comment @cursor review or bugbot run to trigger another review on this PR

@katzino katzino changed the title fix: missing url param on page view [internal] docs: missing url param on page view [internal] Oct 16, 2025
@katzino katzino merged commit fa5e30c into master Oct 16, 2025
11 checks passed
@katzino katzino deleted the fix/segment-page-view-url branch October 16, 2025 08:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working. t-web Issues with this label are in the ownership of the web team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants