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

Performance improvement - Cache #4

Open
bpinto opened this issue Apr 20, 2013 · 7 comments
Open

Performance improvement - Cache #4

bpinto opened this issue Apr 20, 2013 · 7 comments

Comments

@bpinto
Copy link
Owner

bpinto commented Apr 20, 2013

There's no need to download the json file every time hatmaker is used.
Hatmaker should make proper use of caching mechanisms such as etags.

@hzlzh
Copy link

hzlzh commented Apr 24, 2013

JSON cache file is really needed in this workflow.
Also I'll improve the github download link in JSON API for all the users.

@bpinto
Copy link
Owner Author

bpinto commented Apr 24, 2013

Yep, I know but I had to do a lot of code refactor as I had written a lot of junky to get the workflow released. :p

I'll be working on a feature that will as a side effect make this workflow faster. And then I'll work on caching mechanisms.

What do you mean by improve the github download link?

@hzlzh
Copy link

hzlzh commented Apr 24, 2013

Nice, wait for your next faster version, and then maybe I will push your workflow manager on the home page of AlfredWorkflow.com together with my next small workflow Workflow Searcher.

About github download link i've just mentioned is a improvement of the current API, for now, some workflow developers did not add a github download link for their workflows, but other links link Dropbox public link, Droplr link or release page link which is not friendly with the download feature in your workflow manager.
So let's make it better to manage workflows. :)

@hzlzh
Copy link

hzlzh commented Apr 24, 2013

The Github JSON link will be synced with Site JSON link every 24 hours. Still the Github json link is recommended like your current usage in hatmaker

-- updated
API JSON changed here:
hzlzh/AlfredWorkflow.com#4 (comment)

@bpinto
Copy link
Owner Author

bpinto commented Apr 24, 2013

That's great! I'll be working on some server side features also, that I would like to get integrated with alfredworkflow.com. Let's talk about it soon! :)

@hzlzh
Copy link

hzlzh commented Apr 25, 2013

Nice.
BTW, I've released the Workflow Searcher with the API file cached locally, check it here:
https://github.com/hzlzh/Alfred-Workflows/

@bpinto
Copy link
Owner Author

bpinto commented Apr 26, 2013

Nice! We are following different approaches but I like what you did! :)

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

No branches or pull requests

2 participants