cmintey/wishlist

Bug: Wish URL overwritten to bad URL after extraction

Closed this issue · 3 comments

Describe the bug

When attempting to add certain item URLs the URL is overwritten after extraction runs, causing the URL to be invalid. There doesn't appear to be a way to fix the URL after this happens either.

To Reproduce

  1. Open the dialog to add a new item to a wishlist
  2. Enter the URL https://www.rei.com/gift-card/purchase
  3. Observe extraction runs and the URL is changed to https://www.rei.com/null
  4. Attempt to change the URL back to https://www.rei.com/gift-card/purchase, observe extraction runs again and it is reset back to https://www.rei.com/null

Expected behavior

  1. The URL after extraction shouldn't have a null in it - I'm guessing that's an unset variable that failed to extract properly because of the site
  2. The URL should (IMO) "stick" the second time it's set, and not attempt to rerun extraction on the page. (If you want to preserve the ability to re-run the extraction logic, maybe add a refresh button/icon to the right of the Item URL text box for that?)

Screenshots

Screenshot 2024-07-23 at 10 40 20

Desktop (please complete the following information):

  • OS: MacOS 14.5
  • Browser: Firefox 128.0

Interesting. We try to resolve the url by using some opengraph properties when possible. In this case, it seems the issue actually lies on REI's website. I don't know how common this is, but I agree, there should be a way to override this
image

Oh interesting. I'll try to send them a support ticket for that, just to try to fix it on both sides.

The fix works great, thanks @cmintey for all your hard work. 🏆