🔨 Refactored 🧠 Overmind Hacktober | /app/pages/Sandbox/Editor/Workspace/Files/DirectoryEntry/index.js

[malwilley]

What kind of change does this PR introduce?

Refactor /app/pages/Sandbox/Editor/Workspace/Files/DirectoryEntry/index.js to use TS/hooks.

@Saeris @christianalfoni

What is the current behavior?

#2621

What steps did you take to test this? This is required before we can merge.

1. Checked sandbox file directory locally.
2. Ran yarn lint && yarn typecheck

Checklist

• Documentation N/A
• Testing
• Ready to be merged
• Added myself to contributors table

[vercel]

This pull request is being automatically deployed with ZEIT Now (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://zeit.co/codesandbox/codesandbox-client/bm5tajk2e
🌍 Preview: https://codesandbox-cl-git-fork-malwilley-2621-refactor-director-c254e8.codesandbox1.now.sh

[christianalfoni]

Hi there!

Great stuff, though there was a tiny bug there. Have added a PR to your branch which you have to merge in :)

malwilley#1

[christianalfoni]

The following has been tested:

• Create files
• Create directories
• Automatic fork by creating directory
• Delete files
• Delete directory
• Move file
• Validation indication
• Upload files

[malwilley]

@christianalfoni I'm very sorry I somehow missed your comments before this was merged! Thanks for looking at this and finding that bug. Do you want to make a PR to master for that fix?