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

Fix bugs in 3 files #3

Merged
merged 3 commits into from
Apr 1, 2012
Merged

Fix bugs in 3 files #3

merged 3 commits into from
Apr 1, 2012

Conversation

apizer
Copy link
Member

@apizer apizer commented Mar 31, 2012

extraAnswerEvaluators.pl: fix bug that Perl 5.14 flags
contextCurrency.pl: fix bug that Perl 5.14 flags
ImageGenerator.pm: change dir and file permission of tmp images so that a WeBWorK admin can run remove_stale_images. Currently you have to be root to run this.

wwadmin and others added 3 commits March 31, 2012 17:31
mgage added a commit that referenced this pull request Apr 1, 2012
@mgage mgage merged commit 22c55c0 into openwebwork:master Apr 1, 2012
@mgage
Copy link
Sponsor Member

mgage commented Apr 1, 2012

Hi Arnie,

I think I'm starting to understand this pull request thing. Thanks for your submissions.

Could you go to openwebwork/webwork2
and look at the pull request from ionparticle (John Hsu at UBC). He is adding webservice commands
that will make it possible to interact with Canvas and some of the other LTE Blackboard like products.
(Possibly Bill Wheeler will get this done as well but it's nice to have backup.)

See what you think of his solutions, and my comment and perhaps make some of your own.

If we are going to get a lot of people developing collaboratively this kind of discussion is
what we'll need to have happen.

Take care,

Mike

On Mar 31, 2012, at 5:49 PM, Arnold Pizer wrote:

extraAnswerEvaluators.pl: fix bug that Perl 5.14 flags
contextCurrency.pl: fix bug that Perl 5.14 flags
ImageGenerator.pm: change dir and file permission of tmp images so that a WeBWorK admin can run remove_stale_images. Currently you have to be root to run this.

You can merge this Pull Request by running:

git pull https://github.com/apizer/pg master

Or you can view, comment on it, or merge it online at:

#3

-- Commit Summary --

  • Fix bug that Perl 5.14 flags
  • Fix bug the Perl 5.14 flags
  • Set file and directory permission so that a WeBWorK admin can run remove_stale_images to remove old tmp images. Currently you have to be root to run this.

-- File Changes --

M lib/WeBWorK/PG/ImageGenerator.pm (2)
M macros/contextCurrency.pl (2)
M macros/extraAnswerEvaluators.pl (4)

-- Patch Links --

https://github.com/openwebwork/pg/pull/3.patch
https://github.com/openwebwork/pg/pull/3.diff


Reply to this email directly or view it on GitHub:
#3

mgage added a commit that referenced this pull request Sep 7, 2012
make help links produce knowls
mgage pushed a commit that referenced this pull request May 19, 2016
Create ability to add new units to problem before calling NumberWithU…
drgrice1 pushed a commit to drgrice1/pg that referenced this pull request Jan 2, 2022
Add an option to the graph tool to not show the coordinate hints in the lower left corner.
pstaabp pushed a commit that referenced this pull request Jan 4, 2022
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 this pull request may close these issues.

2 participants