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.

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

Not working anymore on FF or Chrome. Price always shows "$NaN / night, including fees" #18

Open
RayBB opened this issue Feb 22, 2022 · 15 comments

Comments

@RayBB
Copy link

RayBB commented Feb 22, 2022

Title says it

But also thank you so much for making this awesome extension @davidsawyer . It has really been a big help and it's a shame airbnb doesn't do this by default.

Cheers

@davidsawyer
Copy link
Owner

davidsawyer commented Feb 22, 2022 via email

@RayBB
Copy link
Author

RayBB commented Feb 22, 2022

Here are a few examples:

However, it happens for pretty much all the listing.
I looked in the dev console and didn't see any errors and I've also tried disabling just about every other extension.
One other factor that may be causing it is that I'm on relatively crappy internet so maybe the high ping and slow page loads mean your extension is looking for things on the page before it loads? Maybe throttling your connection to slow3g will reproduce it? https://www.browserstack.com/guide/how-to-perform-network-throttling-in-chrome
Let me know if there is anything else I can do to help debug it.

@davidsawyer
Copy link
Owner

davidsawyer commented Feb 22, 2022 via email

@The1stSword
Copy link

I believe this is happening is because AirBnB no longer shows the taxes until you hit the Reserve button. So a solution might be to give a user an option to put in a default tax amount for all searches or remove the taxes in the calculation but that kind of defeats the whole purpose of this extension. Unless there is another way?

@davidsawyer
Copy link
Owner

davidsawyer commented Mar 3, 2022 via email

@RayBB
Copy link
Author

RayBB commented Mar 3, 2022

Congrats on the move and the twins! Don't worry too much about the tax thing if it puts too much of a tax on a time of big change in your life :)

@davidsawyer
Copy link
Owner

davidsawyer commented Mar 3, 2022 via email

@davidsawyer
Copy link
Owner

Progress! I did all this programmatically. Now I just need to make it more robust.
Screen Shot 2022-03-11 at 12 03 19 AM

@jmfulgham
Copy link

Hey there, any update on this fix? Thanks again for this resource!

@davidsawyer
Copy link
Owner

Hey, I'm still working on it! Short on time these days, but I promise it's coming soon! (Barring any unforeseen Airbnb changes, of course 😉)

@Mohamed3on
Copy link

hoping you get a fix working soon @davidsawyer :)

@davidsawyer
Copy link
Owner

Thanks for everyone's patience. Here's a little progress gif for you all!
progress

@davidsawyer
Copy link
Owner

Alright team, anyone wanna beta test this rough solution? I have some rough corners still in need of smoothing over.

If you're down to test it, download the v1-beta branch, build it, and run it in your browser. The readme instructions should be able to get you there, but let me know if you have issues.

Let me know in this thread how it works and if/when you notice bugs!

Note that I'm planning on adding a more graceful loading indicator of some kind when it re-fetches the tax info in the background when the user changes dates or something that forces a recalculation. Right now, the updating prices may feel a little awkward and/or laggy.

Thanks!

@RayBB
Copy link
Author

RayBB commented Jun 16, 2022

@davidsawyer just tested v1 beta on firefox and seems to work for some listings but not all.

Appreciate you work fixing this!

Hope this is helpful for debugging :)

Here's the results:

Spoiler

$0 price

https://www.airbnb.com/rooms/46714959?adults=1&children=0&infants=0&check_in=2022-09-14&check_out=2022-09-21&federated_search_id=63abaab7-1f99-43a3-b7b5-14bdc4e63ad4&source_impression_id=p3_1655351916_SBDXMwzzwGuqW5BN

image

Total price is wrong

https://www.airbnb.com/rooms/26532850?adults=1&children=0&infants=0&check_in=2022-09-04&check_out=2022-09-11&federated_search_id=63abaab7-1f99-43a3-b7b5-14bdc4e63ad4&source_impression_id=p3_1655351917_DbfT07H8OunvxQUZ
image

https://www.airbnb.com/rooms/23478156?adults=1&children=0&infants=0&check_in=2022-09-04&check_out=2022-09-11&federated_search_id=63abaab7-1f99-43a3-b7b5-14bdc4e63ad4&source_impression_id=p3_1655351917_QYckz3tbDBX5UT88
image

no price shown at all

https://www.airbnb.com/rooms/4869137?adults=1&category_tag=Tag%3A675&children=0&infants=0&search_mode=flex_destinations_search&check_in=2022-12-25&check_out=2022-12-30&federated_search_id=31de57a9-4feb-41b7-9584-5b8e2c1533fd&source_impression_id=p3_1655351828_%2B4SrNtF58CIscZqa
image

https://www.airbnb.com/rooms/46604828?adults=1&category_tag=Tag%3A675&children=0&infants=0&search_mode=flex_destinations_search&check_in=2022-06-20&check_out=2022-06-25&federated_search_id=31de57a9-4feb-41b7-9584-5b8e2c1533fd&source_impression_id=p3_1655351829_6g75R09oojtS2d2m

image

@davidsawyer
Copy link
Owner

Incredibly valuable bug reports. I'll dig into these soon!

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

No branches or pull requests

5 participants