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

Save current notebook as part of ottr::export() #474

Closed
dbroockman opened this issue Jan 31, 2022 · 3 comments · Fixed by #675
Closed

Save current notebook as part of ottr::export() #474

dbroockman opened this issue Jan 31, 2022 · 3 comments · Fixed by #675
Labels
enhancement New feature or request
Milestone

Comments

@dbroockman
Copy link

Is your feature request related to a problem? Please describe.
When students run ottr::export(), it creates a Zip file of the last saved version of the notebook, which might be out of date. Even though I tell students to manually save before running export, many forget to do so.

Describe the solution you'd like
When ottr::export() runs, have it save the current version of the notebook first.

Describe alternatives you've considered
I've told students to manually save first, but many forget.

Additional context
Many students then complain about their grades being wrong because the last answer or two doesn't show up.

@dbroockman dbroockman added the enhancement New feature or request label Jan 31, 2022
@stale
Copy link

stale bot commented Apr 2, 2022

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the wontfix This will not be worked on label Apr 2, 2022
@stale stale bot removed the wontfix This will not be worked on label Apr 2, 2022
@chrispyles chrispyles removed the pinned label May 24, 2022
@chrispyles
Copy link
Member

chrispyles commented Jul 19, 2023

Opened ucbds-infra/ottr#20 which will be included in the next version of ottr.

After this is available, Otter Assign will need to be updated to pass the argument when applicable.

@chrispyles
Copy link
Member

This is included in ottr v1.4.0 which is on it's way to CRAN.

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

Successfully merging a pull request may close this issue.

2 participants