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

Feature/associate expenselog to paymentmethod #157

Closed

Conversation

WanjikuMac
Copy link
Contributor

Fixes #113

Proposed Changes

  • Associate the expense log to payment method

Copy link
Member

@zacck-zz zacck-zz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome work on this @WanjikuMac just one change and this one goes over the hill

@@ -0,0 +1,15 @@
defmodule Spender.Repo.Migrations.AssociateExpenselogToLogcategory do
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This may need a module doc

@zacck-zz
Copy link
Member

zacck-zz commented Aug 4, 2018

@WanjikuMac please rebase this on develop

@WanjikuMac WanjikuMac force-pushed the feature/associate_expenselog_to_paymentmethod branch from 22c106e to e83e684 Compare August 4, 2018 14:16
@WanjikuMac WanjikuMac closed this Aug 6, 2018
@WanjikuMac WanjikuMac deleted the feature/associate_expenselog_to_paymentmethod branch August 6, 2018 11:39
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