Task/WP-726: Mutation hook: Mkdir

[jalowe13]

Overview

Updating hooks to use React Query mutations with Typescript. This mutation covers making a project directory.

Related

• WP-726

Changes

• useMkDir.js to useMkDir.ts
• Identify types of payloads
• Wraps dispatch actions inside a mutation

Testing

1. Navigate to where you would like to create a directory
2. Press the add button and the folder selection
3. Name your folder
4. Create folder

UI

[codecov]

Codecov Report

Attention: Patch coverage is 77.14286% with 16 lines in your changes missing coverage. Please review.

Project coverage is 72.88%. Comparing base (e5c17d8) to head (d2f0d08).
Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
client/src/hooks/datafiles/mutations/useMkdir.ts	77.14%	16 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #997      +/-   ##
==========================================
- Coverage   72.88%   72.88%   -0.01%     
==========================================
  Files         534      534              
  Lines       33455    33491      +36     
  Branches     2981     2984       +3     
==========================================
+ Hits        24385    24409      +24     
- Misses       8875     8887      +12     
  Partials      195      195              

Flag	Coverage Δ
javascript	75.75% <77.14%> (-0.02%)	⬇️
unittests	60.44% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines	Coverage Δ
client/src/hooks/datafiles/mutations/useMkdir.ts	77.14% <77.14%> (ø)

[jmcmillenmusic]

It works on my end, but for some reason, the Toast notification says that it failed. Not sure why that's happening, but I can see the new folder and access it, so it looks like it works.

[jalowe13]

@jmcmillenmusic

It works on my end, but for some reason, the Toast notification says that it failed. Not sure why that's happening, but I can see the new folder and access it, so it looks like it works.

Resolved this in 2112858

[jalowe13]

Removed test in 2e853fb
let me know if this test needs to be converted to test for something related to react-query instead

[jarosenb]

LGTM!