goodbyteco / letterboxd-watchlist-picker Goto Github PK

you know the bit

Currently, the backend checks if the query contains "list" to differentiate between username/list/list-title and username/list-title, adding back the /list/ if needed. This causes lists with the word "list" in the actual title to break.

For example, this Weirdo Watchlist is unable to be searched: watchlistpicker.com/?u=hamburgersplash/weirdo-watchlist.

Add a banner to tell you that Letterboxd is down if it is

Right now we say cant be found which is wrong

Its hard to edit and read I should fix

Hi
I've been using watchlist picker on a regular basis for quite some time now and I want to thank you for that!
But it seems that it doesn't work anymore. Whatever the profile or options you pick, the same error message is given (see attached)

Is this related to the API? Unfortunately I'm on my phone I can't open a chrome dev tool

see if scraping https://letterboxd.com/holopollock/watchlist/on/netflix-ca/ works

clowchan/list/every-horror-film-made-from-1895-present/

This fails when when on ignore unreleased I think I'm doing random wrong

The backend should convert all list titles to lowercase

once and for all

The link to the backend is broken.

this has cause more issues I should stop being an bad programmer

The submit button is 1px shorter than the input box on some browsers

Here's a rough sketch of placement and look:

Its a super small thing, but all our other repos are title cased

Noticed this bug in testing, but it's live on the site: if you uncheck "unreleased films" in advanced options, the API returns results. But, if you uncheck "short films" it returns "nothing found". Similarly if you have both "unreleased" and "feature length" unchecked, no results.

I believe this is likely an issue with the backend.

It would be nice if the service could also work on non-watchlist lists, such as this one. It would use the same search box and be combinable with regular watchlists too. A possible way to distinguish would be to use the remainder of the URL after "letterboxd.com" such as:

dancedancedance/list/boxing-films

Or perhaps just the username + title of list:

dancedancedance/boxing-films

The first option is easy to copy/paste, but the latter is easier to type. Obviously both could be included.

This is stupid but could be fun to do like the letterboxd scary and holiday theme thing

I figured we could add this accessibility issue pretty easily and would be worth it

you cant get to it with your keyboard atm

The logging is fine but I can make it better it may also improve performance slightly as I log in the hot path

I saw this happen today Damn

Add EsLint config. Related: #98

Implements #78

Hitting submit multiple times triggers multiple submit events, and the random movie result ends up getting overwritten by the later requests. At best it makes the experience feel kind of buggy, at worst, the input changed and the first result(s) are incorrect. This might also be a problem for people who choose to "double click" on things.

Solution: The submit() method should cancel all previous requests, so only the last one shows up. If possible, the loading bar should reset as well.

if this its possible it would be great

Just a general audit to see if there are any quarks we missed.

We should go over all the microcopy on the site, take inventory, and see where we can improve.

Related: #98

As seen in the image below, the submit button jumps to a new line at really small sizes. Solution: shrink input bar.

In the below screenshot, you'll see the recommended film, Phantom Thread, despite the fact I have already watched it (therefore no longer on my watchlist). I kept the url in the shot so you can see that intersection is on, and which usernames I entered. Timestamp: 2020-11-15 9:04:39 PM (PST).

do we want to add this. https://github.com/marketplace/percy

The search box should accept full URLs for watchlists, lists, and actor pages, as well as the short forms

No noticeable effects on the site. Produced on Chrome.

People seem to complain alot about getting recommend films they can't actually go see. Letterboxd treats unreleased films as either having a release date in the future, or no date at all. Films being released later in the current year cannot be determined, but given that the tool is mostly for people looking to get a random result out if a backlog, I think they will rarely want a in-theaters movie (or even a recent release).

Cutting out all current-year movies and dateless movies will have a fair amount of false-positives. I think they'll be a minority of the film's people will want to see, but nonetheless we should have it be an option to turn on/off in the advanced search. I'm leaning towards opt-in.

• add filter logic to backend
• add switch to advanced search
• update "no films found" text to include reference to no released films, when option is selected