bryantd/letterboxd-feed-wp

Personalized code to replicate my Letterboxd reviews to WordPress

letterboxd-feed-wp's Issues

Add spoiler protection for single-movie posts

Belongs in write_movies_to_wp -- see the comments in that function for some details.

WP REST search (and post) APIs don't allow for searching title only

See https://stackoverflow.com/questions/33131103/wp-rest-api-get-post-by-title

The regrettable answer is to do a broader search and check the titles returned.

Write a little python script to display tags/categories and associated IDs

E.g.:

curl https://popone.innocence.com/wp-json/wp/v2/tags | jq .

You don't need to be logged in for this.

Missing reviews.csv doesn't fail gracefully

Just needs another check on config values, maybe an error handler for opening the CSV.

Write a README.md

Convert to OAuth

I've been messing around with this for a month; I should probably take the time to add OAuth.

Improve WP endpoint discovery

Currently assuming that it's always /wp-json/wp/v2/ but it isn't necessarily. See https://developer.wordpress.org/rest-api/using-the-rest-api/discovery/ for instructions.

Add --start-date and --end-date arguments

Only process reviews for movies watched between those dates.

Make the code comply with flake8

Also consider a pre-commit check w/flake8 and black.

• https://flake8.pycqa.org/en/latest/user/using-hooks.html
• https://ljvmiranda921.github.io/notebook/2018/06/21/precommits-using-black-and-flake8/

AttributeError: object has no attribute 'letterboxd_watcheddate

Weekly posts fail at year boundaries

In write_movies_to_wp_by_week, we sort movies into buckets by year and week, which means weeks spanning years result in bogus bucketing.

Fixable by just going back to the previous Monday (which would also fix part of #25) and iterating over 7 day chunks from there.

Filter out posters from reviews

One of the two fetches isn't filtering out movie posters properly.

Add update check for posting individual reviews

Possible that a review will change and a single review post will need to be updated, so check for that. Should just be part of write_movies_to_wp.

Movies without a rating break the fetch routines

Add date filters to fetchcsv

It takes forever to run the script on a large export even if you just need to fill in a couple of weeks. --start-date and --end-date should work here.

Handle missing categories and tags in the config file

Script should behave gracefully if they don't exist.

Add real exception handling to all the HTTP requests

As per the wp_post function:

            try:
                response = requests.post(
                    f"{wp_post_api}/{post_id}", headers=wp_headers, json=post
                )
            except requests.exceptions as error:
                print(f"ERROR: post failed: {error}")
            if response.status_code != requests.codes.ok:
                print(f"ERROR: post failed w/status code {response.status_code}")

Every use of requests should look more like that.

Add a configuration option allowing for <i> tags instead of my [cite] shortcode

I'm the only human being in the world stubborn enough to write a mini-plugin to make WordPress use cite tags.

--start-date and --end-date options could result in partial week posts

Since I currently implement dates via the SQL SELECT statement, if I use the writeweeks option, I run the risk of missing movies in the weeks containing the start/end date. Probably the fix to this is to do some date math and expand the date range to contain full weeks as needed.

Die gracefully if required config elements are missing