Skip to content

fix(map): do not remove query parameters from results (FIR-1015)#1191

Merged
nickscamara merged 2 commits intomainfrom
mog/fix-map-query-param-handling
Feb 17, 2025
Merged

fix(map): do not remove query parameters from results (FIR-1015)#1191
nickscamara merged 2 commits intomainfrom
mog/fix-map-query-param-handling

Conversation

@mogery
Copy link
Member

@mogery mogery commented Feb 17, 2025

Fixes sitemapOnly's faithfulness to the real sitemap, and also should improve Extract for sites that use URLs like https://example.com/product?id=<xyz> for product pages/anything that needs to be multi-entity extracted.

@mogery mogery requested a review from nickscamara February 17, 2025 09:07
@mogery mogery force-pushed the mog/fix-map-query-param-handling branch 6 times, most recently from 3882dff to dd8cd63 Compare February 17, 2025 09:36
@mogery mogery force-pushed the mog/fix-map-query-param-handling branch from dd8cd63 to ba2fe23 Compare February 17, 2025 09:40
@mogery
Copy link
Member Author

mogery commented Feb 17, 2025

NOTE: test failing is unrelated (f-e error), reported issue to Thomas

@nickscamara nickscamara merged commit 7ecbff3 into main Feb 17, 2025
1 check passed
@mogery mogery deleted the mog/fix-map-query-param-handling branch February 17, 2025 20:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants