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

Add link to download all worksheets as a zip file for accounts=True #7962

Closed
TimDumol mannequin opened this issue Jan 17, 2010 · 5 comments
Closed

Add link to download all worksheets as a zip file for accounts=True #7962

TimDumol mannequin opened this issue Jan 17, 2010 · 5 comments

Comments

@TimDumol
Copy link
Mannequin

TimDumol mannequin commented Jan 17, 2010

This is already present in the codebase introduced by Robert Bradshaw and revised by William Stein, but there is no link to the url for the file.

CC: @jasongrout @williamstein

Component: notebook

Author: Tim Dumol

Reviewer: Mitesh Patel

Merged: sagenb-0.7

Issue created by migration from https://trac.sagemath.org/ticket/7962

@TimDumol TimDumol mannequin added c: algebra labels Jan 17, 2010
@TimDumol TimDumol mannequin assigned aghitza Jan 17, 2010
@TimDumol
Copy link
Mannequin Author

TimDumol mannequin commented Jan 17, 2010

Simple 2 line patch removing the {% if not accounts %} tags.

@TimDumol
Copy link
Mannequin Author

TimDumol mannequin commented Jan 17, 2010

comment:1

Attachment: trac_7962-link-worksheets-zip-file.patch.gz

@TimDumol TimDumol mannequin added the s: needs review label Jan 17, 2010
@TimDumol TimDumol mannequin changed the title Add link to download all worksheets as a zip file Add link to download all worksheets as a zip file for accounts=True Jan 17, 2010
@jasongrout
Copy link
Member

comment:2

#7963 needs to be fixed, probably before this ticket is fixed.

@aghitza aghitza added this to the sage-4.3.2 milestone Jan 17, 2010
@qed777
Copy link
Mannequin

qed777 mannequin commented Jan 19, 2010

Reviewer: Mitesh Patel

@qed777
Copy link
Mannequin

qed777 mannequin commented Jan 25, 2010

Merged: sagenb-0.7

@qed777 qed777 mannequin removed the s: positive review label Jan 25, 2010
@qed777 qed777 mannequin closed this as completed Jan 25, 2010
@qed777 qed777 mannequin mentioned this issue Feb 10, 2010
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants