Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

👀 Hiding Sandpack Tests #768

Closed
1 of 2 tasks
ubeytd opened this issue Feb 26, 2023 · 2 comments · Fixed by #775
Closed
1 of 2 tasks

👀 Hiding Sandpack Tests #768

ubeytd opened this issue Feb 26, 2023 · 2 comments · Fixed by #775

Comments

@ubeytd
Copy link
Contributor

ubeytd commented Feb 26, 2023

Tell us how you think we can improve Sandpack

Providing Context and Use Case

At our school, we want to incorporate coding challenges into our platform to assess our students' coding abilities.

We plan on using sandpack-react to achieve this goal. However, we want to ensure that our students are not just memorizing solutions, but are truly able to think and problem-solve. Therefore, we would like to hide the tests completely from view.

Hidden Test Files

In verbose mode, when a test file fails, a link is displayed to show where the test file failed. However, if the test file is meant to be hidden, it would be great to address this behavior.

Optional Props to Hide Verbose and Watch Buttons

In order to manually run tests with dispatch({type:'refresh'}) and hide test file content in the Test component, we suggest adding showVerbose and showWatch props to the Test component. 🤗

Packages affected

  • sandpack-client
  • sandpack-react
@ubeytd ubeytd added feature:request triage New issues that needs consideration labels Feb 26, 2023
@danilowoz
Copy link
Member

Hey @ubeytd, that would be great! I'm really happy to know you're considering Sandpack in your coding challenges.

I have added your feature request to our internal backlog which we use to prioritize what to build next. We can't make any promises about the timing, but it's on the list! If you'd like to work on this feature with us, please create a pull request and we'll help you along the way.

@danilowoz danilowoz removed the triage New issues that needs consideration label Feb 27, 2023
@ubeytd
Copy link
Contributor Author

ubeytd commented Feb 28, 2023

Awesome! Created PR 🤗

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

Successfully merging a pull request may close this issue.

2 participants