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

Fix/code style fixes #1

Merged
merged 3 commits into from
Apr 20, 2018
Merged

Fix/code style fixes #1

merged 3 commits into from
Apr 20, 2018

Conversation

lucca65
Copy link
Member

@lucca65 lucca65 commented Apr 20, 2018

  • Move chain and wallet modules to their own folder, following elixir practices
  • Generalized usage of get_request/1 post_request/2 and post_request/3
  • Generalized usage of base_url
  • Run elixir format on all files

@lucca65
Improved readability
fd3148a
@lucca65
Several changes on code structure
29d8aa1 
- Move chain and wallet modules to their own folder, following elixir practices
- Generalized usage of get_request/1 post_request/2 and post_request/3
- Generalized usage of base_url
@lucca65 lucca65 added the enhancement New feature or request label Apr 20, 2018
@lucca65 lucca65 requested a review from leordev April 20, 2018 15:30
@leordev leordev merged commit 84c8f9e into master Apr 20, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@leordev leordev Awaiting requested review from leordev

Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@lucca65 @leordev