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

Issue 96: Submit exercises via GUI #112

Merged
merged 2 commits into from
Nov 13, 2018
Merged

Issue 96: Submit exercises via GUI #112

merged 2 commits into from
Nov 13, 2018

Conversation

samWson
Copy link
Contributor

@samWson samWson commented Nov 7, 2018

Fixes an unknown method on ExTonelWriter, giving it a complete in memory file system for
writing exercise source code.

Fixes an unknown method on ExTonelWriter, giving it a complete in memory file system for
writing exercise source code.
@samWson
Copy link
Contributor Author

samWson commented Nov 7, 2018

Fixes #96.

I managed to sucessfully submit an exercise from the GUI with the code that uses the http PATCH method from a Windows 10 machine.

It deserves testing from a fresh image to be sure, but I think we can finally call this issue done. I'll leave this PR open for you to review if you have the time, and then I'll merge later.

@@ -47,7 +47,7 @@ ExercismSubmit class >> exercise: exerciseId [

{ #category : #internal }
ExercismSubmit >> buildRequest [
| solutionClasses multiPartFormDataEntity solutionId exercise |
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just removing unused local variables.

@bencoman
Copy link
Contributor

bencoman commented Nov 7, 2018

Its cool that you tracked down that problem, but I'm curious why all this is still needed rather than using ExercismSubmit on its own? None of that preceding work seems to be used by ExercismSubmit.
i.e. similar to this

Hope you enjoy your break.

@samWson
Copy link
Contributor Author

samWson commented Nov 8, 2018

@bencoman thanks for spotting that. I'll look into it. I'm more than happy to clean up code we don't need anymore.

Removed code was made redundant when the switch to submitting exercises via HTTP PATCH was made
instead of using OSProcess.
@samWson
Copy link
Contributor Author

samWson commented Nov 12, 2018

Just pushed a new change to remove the dead code. Thanks for spotting @bencoman.

@samWson samWson merged commit 6bdb5b0 into master Nov 13, 2018
@samWson samWson deleted the http-submit-exercise branch November 19, 2018 07:09
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.

None yet

2 participants