#598 refactor code to use Shrinkwraprs and diesel-derive-newtype

Merged
igalic merged 6 commits from refactor/shrinkwraps into master 1 year ago
igalic commented 1 year ago (Migrated from github.com)

Reduce a lot of our own boilerplate with these two crates.
This fixes #597.

Reduce a lot of our own boilerplate with these two crates. This fixes #597.
codecov[bot] commented 1 year ago (Migrated from github.com)
Owner

Codecov Report

Merging #598 into master will increase coverage by 0.09%.
The diff coverage is 16.66%.

@@            Coverage Diff            @@
##           master    #598      +/-   ##
=========================================
+ Coverage    34.5%   34.6%   +0.09%     
=========================================
  Files          67      67              
  Lines        7876    7840      -36     
  Branches     1890    1883       -7     
=========================================
- Hits         2718    2713       -5     
+ Misses       4398    4367      -31     
  Partials      760     760
# [Codecov](https://codecov.io/gh/Plume-org/Plume/pull/598?src=pr&el=h1) Report > Merging [#598](https://codecov.io/gh/Plume-org/Plume/pull/598?src=pr&el=desc) into [master](https://codecov.io/gh/Plume-org/Plume/commit/59023e96023405847b3a91658e0adffecceb993d?src=pr&el=desc) will **increase** coverage by `0.09%`. > The diff coverage is `16.66%`. ```diff @@ Coverage Diff @@ ## master #598 +/- ## ========================================= + Coverage 34.5% 34.6% +0.09% ========================================= Files 67 67 Lines 7876 7840 -36 Branches 1890 1883 -7 ========================================= - Hits 2718 2713 -5 + Misses 4398 4367 -31 Partials 760 760 ```
elegaanz (Migrated from github.com) reviewed 1 year ago
@@ -140,26 +140,12 @@ impl FromId<PlumeRocket> for Reshare {
let res = Reshare::insert(
&c.conn,
NewReshare {
Plume_migration_agent commented 1 year ago

I think you can even replace this block with just &act.announce_props.object_link::<Id>()?, it was only here to be able to convert the Id to a String (and same for other similar blocks).

I think you can even replace this block with just `&act.announce_props.object_link::<Id>()?`, it was only here to be able to convert the Id to a String (and same for other similar blocks).
trinity-1686a approved these changes 1 year ago

Reviewers

trinity-1686a approved these changes 1 year ago
The pull request has been merged as 8c59c822b6.
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.