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

Save load requests #29

Merged
merged 29 commits into from
Mar 15, 2016
Merged

Save load requests #29

merged 29 commits into from
Mar 15, 2016

Conversation

javaguirre
Copy link
Contributor

With this functionality we can save requests (100 for now, but this will be configurable in the future). We also have a recent requests feature where the most recent requests are stored.

@ddavison Can you check if It's working for you too?

Any opinion or criticism, here I am!

Note: the Unittests are passing, but sometimes there is some kind of timeout problem with Travis CI, we might need to change this configuration.

I added two config options, collections path and recent requests limit
and its defaults
I also added a TODO to fix atom.config to be passed to the view, now
It doesn't work
I also added an empty spec file for persist
With events we separate logic in differents parts of our module. For
now the loading button
@javaguirre
Copy link
Contributor Author

I fixed the method highlighting when loading requests, I think we're ready to merge, I'll try to fix the Travis environment this week.

@ddavison Please let me know if you find anything, otherwise we could merge and create a new version. :-)

@javaguirre
Copy link
Contributor Author

I've spent some time on the Travis issue, but I haven't found how to fix It yet. I'll vote for merging the PR and create the new GIF for the Readme, I can do It if you're ok with It @ddavison.

@ddavison
Copy link
Owner

yea, sorry - i just got a new computer, so still customizing it, and getting all my workspaces back up ;)

@ddavison
Copy link
Owner

i'd also like to implement #30 in this branch

ddavison added a commit that referenced this pull request Mar 15, 2016
@ddavison ddavison merged commit 6152c94 into master Mar 15, 2016
@ddavison
Copy link
Owner

going to push the new version now

@ddavison
Copy link
Owner

pushed as new major version. 1.0 finally :P

@javaguirre
Copy link
Contributor Author

Veery nice! :-D

On Tue, Mar 15, 2016, 18:33 Dj notifications@github.com wrote:

pushed as new major version. 1.0 finally :P


You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
#29 (comment)


Javier Aguirre
https://javaguirre.net

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

Successfully merging this pull request may close these issues.

2 participants