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

Creates expense management #166

Merged
merged 24 commits into from
Jul 23, 2020
Merged

Creates expense management #166

merged 24 commits into from
Jul 23, 2020

Conversation

ayush-goyal
Copy link
Member

Changes

Creates expense flow. Users with appropriate permissions can manage the requisition status through expenses in the app. Based on the different stages of the requisition, the appropriate actions will show. Additionally, I implemented more of the Payment, PaymentMethods, and Approval with GraphQL and in the frontend. Most of the work was done in the expense section of the requisition detail page. I also created a new partly received status to signify when only some of the items have arrived. Additionally, there are more status cards on the requisition page as appropriate with regards to the approval, payment, and shipping location.

@ayush-goyal ayush-goyal requested a review from evan10s July 15, 2020 23:56
@ayush-goyal
Copy link
Member Author

Added a few updates:

  • New payments section table, will display all the payments for a requisition
  • Now, payment can be created in multiple statuses
  • Actions section will conditionally show when a requisition is closed or cancelled, it is hidden
  • Manage status action buttons have more explicit text
  • Approval card updated, shows new data format
  • Refactored a lot of the requisition detail code into individual sections

(Note, the autofill payment amount will be done separately, see issue #165 )

expenses/models.py Outdated Show resolved Hide resolved
expenses/schema.py Show resolved Hide resolved
@ayush-goyal ayush-goyal requested a review from evan10s July 20, 2020 16:15
@ayush-goyal ayush-goyal merged commit eb3d9ef into master Jul 23, 2020
@ayush-goyal ayush-goyal deleted the ayush/expenses branch July 23, 2020 02:43
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