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

Correctly update the budgeted amount when using UpdateCategoryForCurrentMonth #25

Closed
wants to merge 117 commits into from
Closed

Correctly update the budgeted amount when using UpdateCategoryForCurrentMonth #25

wants to merge 117 commits into from

Conversation

johandk89
Copy link

@johandk89 johandk89 commented Feb 23, 2021 •
edited
Loading

As seen here the structure updated to be:

{
  "category": {
    "budgeted": 0
  }
}

Fixes #24

Bruno M V Souza added 30 commits July 11, 2018 00:24
Add account type constants
0bfdfcc
Add account representation
f4e4eab
Improve path organization
d344706
Ignore vendor folder
33487cf
Add client
c5a85d3
Add account API tests
5ebd745
Add dep
64daa81
Add user service and tests
3478f7a
Add budget service's GetBudgets support
123d9ad
Add budget service's GetBudgetByID and GetBudgetDeltaByID support
165a1ef
Add budget service's GetBudgetSettingsByID support
54e643b
Add category service's GetCategories support
7cdc1f4
Add category service's GetCategoryByID support
eca9555
Simplify service tests
3870d1b
Add payee service's GetPayees support
b5b4b15
Add payee service's GetPayeeByID support
09fca15
Improve url visibility on service tests
b3b3098
Add payee service's GetPayeeLocations support
bfe22bd
Add payee service's GetPayeeLocationByID support
8e1d373
Add payee service's GetPayeeLocationsByPayeewq support
9c51b7d
Improve entities documentation
49dfe32
Add month service support
34e4932
Add transaction service's GetTransactions support
cf01227
Improve and better document entities
f67485b
Improve budget filtering
86be76b
Add transaction service's GetTransactionsByAccountID support
c38face
Add transaction service's GetTransactionsByCategory support
d0ef960
Add transaction service's GetTransactionsByPayee support
ab58c3e
Add transaction service's GetTransaction support
c504961
Add transaction service's GetScheduledTransactions support
365cfb0
Bruno M V Souza and others added 27 commits December 20, 2018 18:56
Add ability to fetch a category for a given month
cbbee00 
Implements wrapper for https://api.youneedabudget.com/v1#/Categories/getMonthCategoryById
Add ability to update a category for a given month
e396d41 
Implements wrappers for https://api.youneedabudget.com/v1#/Categories/updateMonthCategory
Merge pull request brunomvsouza#18 from brunomvsouza/api-documentatio…
76d82d5 
…n-sync

Synchronizes latest API changes
Fix godoc badge link
43c11e4
Add support for delta requests while retrieving accounts
1f17819
Removing budget.Filter in favor of api.Filter
43c137a
Fix server knowledge data type
8ec5800
Add support for delta requests while retrieving categories
afde17d
Add support for delta requests while retrieving months
720543b
Add support for delta requests while retrieving payees
2954acd
Allow PATCH requests
4163b3f
Add support for bulk transaction updates
ca013e9
Merge pull request brunomvsouza#19 from brunomvsouza/api-updates
ed5f2af 
API updates
Fix maitained badge
907bd17
Add Gitlab CI support
da1d332
Fix CI badge
1ce970f
Merge branch 'fix-badge' into 'master'
d6331a5 
Fix CI badge

See merge request bs/ynab.go!20
Using gitlab's badages
44731d7
Merge branch 'fix-badge' into 'master'
f2cf633 
Using gitlab's badages

See merge request bs/ynab.go!21
Remove unecessary badge
17cc451
Merge branch 'fix-badge' into 'master'
a08d11c 
Remove unecessary badge

See merge request bs/ynab.go!22
Use Go Report Card based on LAB
0815e9f
Explicitly use golang's Debian Stretch image to avoid build errors
581752c 
With the advent of Debian Buster, go's base images started to be build using Debian Buster instead of Debian Stretch, which broke the build pipeline.
@iillexial
Support ID in TransactionPayload
47de57c
Replace gitlab-ce pipeline with travis-ci
6d1b740
@johandk89
Correctly update the budgeted amount when using UpdateCategoryForCurr…
66840b0 
…entMonth
@johandk89
Updated failing tests
6691e60
@coveralls
Copy link

Pull Request Test Coverage Report for Build 705

  • 1 of 1 (100.0%) changed or added relevant line in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+1.07%) to 95.573%
Totals Coverage Status
Change from base Build 373: 1.07%
Covered Lines: 626
Relevant Lines: 655
💛 - Coveralls

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

UpdateCategoryForCurrentMonth doesn't update the Budgeted amount
7 participants
@johandk89 @coveralls @brunomvsouza @dwburke @appleboy @hoenn @iillexial