Skip to content
This repository has been archived by the owner on Dec 4, 2023. It is now read-only.

fix(cacheMethods): there should be executeRaw in defaultMutationMethods #660

Merged
merged 2 commits into from
Dec 3, 2023
Merged

fix(cacheMethods): there should be executeRaw in defaultMutationMethods #660

merged 2 commits into from
Dec 3, 2023

Conversation

Hazmi35
Copy link
Contributor

@Hazmi35 Hazmi35 commented Dec 3, 2023

  • fix(cacheMethods): it is executeRaw even if you execute executeRawUnsafe
    The params.action would be executeRaw even if you execute executeRawUnsafe
  • fix(cacheMethods): add executeRawUnsafe too for compatibility
    As mentioned in the issue, I'm not sure if Prisma v4 params.action is executeRaw even on executeRawUnsafe. For compatibility, let's add both

The params.action would be executeRaw even if you execute executeRawUnsafe
As mentioned in the issue, I'm not sure if Prisma v4 params.action is executeRaw even on executeRawUnsafe. For compatibility, let's add both
@Hazmi35
Copy link
Contributor Author

Hazmi35 commented Dec 3, 2023

fixes #661

Copy link

codecov bot commented Dec 3, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (e6073f3) 97.20% compared to head (2b3efdf) 97.22%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #660      +/-   ##
==========================================
+ Coverage   97.20%   97.22%   +0.01%     
==========================================
  Files           2        2              
  Lines         179      180       +1     
  Branches       34       34              
==========================================
+ Hits          174      175       +1     
  Misses          5        5              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@Asjas Asjas merged commit e6ed17b into Asjas:main Dec 3, 2023
15 checks passed
@Hazmi35
Copy link
Contributor Author

Hazmi35 commented Dec 4, 2023

Hi! Thanks for the quick merge. What's your plan on releasing this?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants