[Feature] Add retirement dates for sets already retired #37
Reference in New Issue
Block a user
Delete Branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Current implementation of retirement date pulls data from a source that tracks upcoming retirements.
This means that when adding a set to wishlist which is already retired, retirement date will not be picked up, and field will be empty.
Feature request is to find a data source that has retirement dates for already retired sets, and add this date to the wishlist.
Brickset and Rebrickable seems to have "to"/"exit"-dates for retired sets.
Additional: Colorcoding the retirement dates: ie. Yellow for upcoming retirements, red for already retired sets.
Example set already retired: Set 10185:
https://brickset.com/sets/10185-1/Green-Grocer
https://rebrickable.com/sets/10185-1/green-grocer/
Retirement date is unfortunately not part of the rebrickable api, which is why it is implemented differently.
Ahh, Thought it might only be the “future retirement” that wasn’t there, from how I read the initial discussion on this feature.
So it would need to implement another source (for instance Brickset, which I dont know about, so not even sure if it has api or it would work).
As a possible workaround, maybe make it so retirement date, once it has been pickee up by current implementation, is not deleted from BrickTracker again, once it is removed from the spreadsheet.
That Way, it would atleast Track previously reirement of sets you added before they were retired.
Not sure if it already Works this Way now.
Rebrickable gives this info in their API:
Which unfortunately isn't a lot compared to what's available on their site.
It looks like Brickset does offer a launch and retirement date in their API, but getting an API key is a manual application and not automatic, which makes it harder.
Saving previous dates could work. I will keep the issue open and add it as an idea for future improvements.
Add retirement dates for sets already retiredto [Feature] Add retirement dates for sets already retired@jonaskp
I might have found a solution to this problem. If I'm able to source release/retirement dates (mix of Bricklink/Rebrickable/Brickset/Peeron/LEGO.com/brickipedia.fandom info), I'm thinking of adding a second container that runs a very simple FastAPI container which can query a set number (eg. http://localhost:8000/21005) and it then returns a json object with data needed.
This way I can update the secondary image more often with new data. I'm also thinking of adding data like official LEGO description, tags, age rating, MSRP etc to this data. But I'm trying not to bundle it with BrickTracker to avoid constant update of the container image.
BrickTracker could check if this container is available and automatically use this info on
/wishesand set-details could also use this info if needed.@FrederikBaerentsen I created an account to request an option for automatic release MSRP filling for the "Price" field because I'd rather have that than nothing, maintaining the ability to edit for difference when purchase price is actually different from MSRP. Seems like this may be "easily" (I don't think any feature is easy, I'm not trying to imply I'm one of those people) added to this particular feature if you're planning to have the MSRP available anyway. Would you like me to make a new feature request or just leave it tacked on to this one?
@fatherfork Let's keep it in this feature request, as it's similar.
As mentioned before, the issue is where to source the data from. Rebrickable doesn't include it in their API, so we need it elsewhere from. It is a hassle for everyone to have their own API key for Brickset.
After i get done with #68 and #69, I'll look at this one.