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

Check for permissions directory #566

Open
elinw opened this issue Nov 12, 2016 · 2 comments
Open

Check for permissions directory #566

elinw opened this issue Nov 12, 2016 · 2 comments
Labels
swirl 2023 Something we should consider when redesigning swirl

Comments

@elinw
Copy link

elinw commented Nov 12, 2016

Because I am running my class on RStudioServer each student has his or her own folder for progress. Because you need root to create those folders, what I have done is to create the folders for each user using a naming scheme.
That all works great, but if a student types in the wrong name (or uses the wrong case etc.) the gzip error shows up.
#60

I wonder if when this returns FALSE
https://github.com/swirldev/swirl/blob/master/R/menu.R#L33
There could be a message saying to try again or that the user may not have permission to create a directory.

@seankross
Copy link
Member

I like this idea. Are you specifying their data directories using: swirl_options() ?

@elinw
Copy link
Author

elinw commented Nov 17, 2016

No I have a cron job that runs that creates the directories for each user using their usernames. They also have their home folders for normal RStudio work. But i really prefer having a place they don’t have direct access to for storing the results.

Elin

On Nov 16, 2016, at 6:16 PM, Sean Kross notifications@github.com wrote:

I like this idea. Are you specifying their data directories using: swirl_options() ?


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub #566 (comment), or mute the thread https://github.com/notifications/unsubscribe-auth/AAuEffT6C5MIX7jYoY2pmQcIIDS_EoR5ks5q-47dgaJpZM4Kwje0.

@cansavvy cansavvy added the swirl 2023 Something we should consider when redesigning swirl label Feb 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
swirl 2023 Something we should consider when redesigning swirl
Projects
None yet
Development

No branches or pull requests

3 participants