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.

Sign up for GitHub

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

Jump to bottom

Refactoring Redis step into separate function #15

Open
gmanroney opened this issue Jan 3, 2018 · 0 comments
Open

Refactoring Redis step into separate function #15

gmanroney opened this issue Jan 3, 2018 · 0 comments
Labels
enhancement

Comments

@gmanroney
Copy link
Owner

Refactoring of redis step in each exhange function into its own separate function.
gmanroney pushed a commit that referenced this issue Jan 3, 2018
Changes for #14 (consistent messaging) and #15 (moving redis to its o…
8311d8c 
…wn function)
gmanroney pushed a commit that referenced this issue Jan 4, 2018
Fixed minor error raised in grunt because of changes for #15
e8fc795
gmanroney pushed a commit that referenced this issue Jan 4, 2018
Further refinement of code as part of #15
302e3ec
gmanroney pushed a commit that referenced this issue Jan 4, 2018
Further improvements as part of #15; made client connection a global …
e5159fe 
…variable so it does not need to be passed down as an argumentin the functions
gmanroney pushed a commit that referenced this issue Jan 4, 2018
Further rationalization of code as part of #15. Simplified some of th…
70cfbeb 
…e logging function used in diagnostics
gmanroney pushed a commit that referenced this issue Jan 4, 2018
Further changes as part of #15. Also changed code for how client work…
d45dc8a 
…s to make it more reusable
gmanroney pushed a commit that referenced this issue Jan 4, 2018
Further cleanup as part of #15
fb0e7e4
gmanroney pushed a commit that referenced this issue Jan 4, 2018
Removing unrequired file as part of #15
4dd4e1a
gmanroney pushed a commit that referenced this issue Jan 8, 2018
Minor changes to cleanup code and remove un-required console.log for #14
fd9c5bf 
 and #15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@gmanroney