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

Fixing some achievements (third_time, persistence, super_persistence) #1324

Merged
merged 1 commit into from
Apr 16, 2021

Conversation

robert-marik
Copy link
Contributor

@robert-marik robert-marik commented Apr 15, 2021

Fix for three achievements. They do not work as advertised. Without this fix it would be possible to earn achievement "Solve a problem on the third submission" by entering correct answer followed by two incorrect answers to the same question. Similarly for other two achievements.

This should be installed after #1319

@Alex-Jordan Alex-Jordan merged commit 817a19f into openwebwork:develop Apr 16, 2021
@Alex-Jordan
Copy link
Contributor

Ugg. So I wasn't paying attention and I see that I merged this into develop. Also #1321. What is the right thing to do right now to get these into 2.16 without causing a mess with `develope?

@drgrice1
Copy link
Member

I will take care of it.

@drgrice1
Copy link
Member

I will presume that all are intended for the release.

@drgrice1
Copy link
Member

At least of the ones just merged that is.

@Alex-Jordan
Copy link
Contributor

imho they should all go into the release. They fix things that were not working as they should have been. And they are so small scale that there's no need for more testing.

@drgrice1
Copy link
Member

Yeah, the two branches are synchronized now.

@Alex-Jordan
Copy link
Contributor

Thanks. I remembered to change to WeBWorK-2.16 yesterday with #1317, but forgot about it today.

@robert-marik
Copy link
Contributor Author

Thank you for including to 2.16.

@robert-marik robert-marik deleted the thirdTime branch April 29, 2021 20:24
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.

4 participants