Skip to content
This repository has been archived by the owner on Nov 27, 2018. It is now read-only.

Implement Citadel Asset Listings #21

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mglinski
Copy link

Implements Citadel Information Lookups using the stop.hammerti.me.uk structures api.

@whinis
Copy link
Owner

whinis commented Jul 16, 2017

Hey thanks for this, but I would personally like it if it didn't download a 1Mb json file essentially every request. Before I would merge this I would want some sort of caching on this beyond storing it for the current request. That way its only downloaded once a day or month perhaps.

As far as I can see there really isn't a nice "public" caching function but within the next couple of weeks there should be one for my ESI rewrite.

@whinis
Copy link
Owner

whinis commented Aug 13, 2017

Not sure if you are still monitoring this however I recently added the first part of the ESI update which includes a new caching class almost specifically for this reason.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants