Skip to content

Conversation

@morremeyer
Copy link
Member

Resolves #123.

BREAKING CHANGE: Resource IDs are now UUIDs. You need to delete your
existing database before this version will start up.

@codecov
Copy link

codecov bot commented May 21, 2022

Codecov Report

Merging #170 (f09b8fe) into main (d7a6fc9) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #170      +/-   ##
==========================================
+ Coverage   99.04%   99.05%   +0.01%     
==========================================
  Files          16       16              
  Lines        1147     1161      +14     
==========================================
+ Hits         1136     1150      +14     
  Misses          7        7              
  Partials        4        4              
Impacted Files Coverage Δ
internal/httputil/request.go 100.00% <ø> (ø)
internal/models/account.go 100.00% <ø> (ø)
internal/models/transaction.go 78.57% <ø> (ø)
internal/controllers/account.go 100.00% <100.00%> (ø)
internal/controllers/allocation.go 97.77% <100.00%> (+0.05%) ⬆️
internal/controllers/budget.go 100.00% <100.00%> (ø)
internal/controllers/category.go 100.00% <100.00%> (ø)
internal/controllers/envelope.go 100.00% <100.00%> (ø)
internal/controllers/transaction.go 100.00% <100.00%> (ø)
internal/httputil/error.go 100.00% <100.00%> (ø)
... and 2 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d7a6fc9...f09b8fe. Read the comment docs.

Resolves #123.

BREAKING CHANGE: Resource IDs are now UUIDs. You need to delete your
existing database before this version will start up.
@morremeyer morremeyer merged commit 0e8f00e into main May 21, 2022
@morremeyer morremeyer deleted the feat/uuids branch May 21, 2022 13:32
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.

Use UUIDs for all resources

2 participants