codesandbox-client icon indicating copy to clipboard operation
codesandbox-client copied to clipboard

Fixed Form Menu item for Read user

Open Akash-Ramjyothi opened this issue 3 years ago • 4 comments

What kind of change does this PR introduce?

Read users don't have the ability to fork. Clicking this menu option leads to an error notification, so hiding it. Left the option in the command palette if you seek it out - it will still give you an error notification. Not sure if there's a way to show it but disable.

Checklist

  • [x] Documentation
  • [x] Testing
  • [x] Ready to be merged
  • [ ] Added myself to contributors table

Akash-Ramjyothi avatar Apr 15 '22 13:04 Akash-Ramjyothi

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit ae17816dc56832fd1f91804a5ec9f4b398b57c07:

Sandbox Source
Notifications Test Configuration

codesandbox-ci[bot] avatar Apr 15 '22 13:04 codesandbox-ci[bot]

Build for latest commit https://github.com/codesandbox/codesandbox-client/pull/6638/commits/ae17816dc56832fd1f91804a5ec9f4b398b57c07 is at https://pr6638.build.csb.dev/s/new.

lbogdan avatar Apr 15 '22 13:04 lbogdan

@lbogdan sir please review my PR.

Akash-Ramjyothi avatar May 09 '22 18:05 Akash-Ramjyothi

Hmm, I don't think this one's true. If you have read access, you should still be able to fork because that's how you get your own version to work on it. What kind of error are you getting? Because that would be abug.

CompuIves avatar Jun 02 '22 13:06 CompuIves