Skip to content

Conversation

JossyYila
Copy link
Contributor

  • Kind: feature

This PR implements backend of the feature requests #331.
@erikmd I would like to notify that the server returns a 500 error when you want to connect with a teacher token.

Checklist

Note to maintainers

  • Read this wiki page
  • Make sure the PR has a milestone
  • Assign yourself before merging
  • We can squash-merge 1-commit PRs (use a header with a conventional-commit type, add a footer with Fix #… if need be)

@erikmd erikmd changed the title feat(src & demo-repository): add back for multi-part exercice feat: Add support for multi-part exercices Jun 21, 2022
@erikmd erikmd added the kind: feature New user-facing feature. label Jul 5, 2022
@@ -455,6 +455,7 @@ let update_tabs meta exo ans =
update_answer_tab ans.Answer.solution

let () =
print_string ("Test Show exo stV : \n");
Copy link
Contributor Author

Choose a reason for hiding this comment

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

remove or comment this print

@@ -334,7 +334,7 @@ module Conversions (Json: JSON_CODEC) = struct
| Exercise_index None ->
get ["exercise-index.json"]

| Exercise (Some token, id) ->
| Exercise (Some token, id) -> print_string ("Api_multipart_2_ : \n");
Copy link
Contributor Author

@JossyYila JossyYila Jul 11, 2022

Choose a reason for hiding this comment

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

remove or comment the print_string

JossyYila and others added 15 commits July 12, 2022 10:45
Still TODO @JossyYila:

- don't hardcode the demo-multi.json anymore
- don't generate the demo-multi/*.json anymore
- remove the warnings *after* understanding the bugs they helped spotting
  typically:

File "src/app/learnocaml_exercise_main.ml", line 229, characters 56-61:
229 |   retrieve (Learnocaml_api.Exercise_index tok) >>= fun (index,l)
->                                                             ^^^^^
Warning 27 [unused-var-strict]: unused variable index.

File "src/app/learnocaml_exercise_main.ml", line 229, characters 62-63:
229 |   retrieve (Learnocaml_api.Exercise_index tok) >>= fun (index,l)
->                                                                  ^
Warning 27 [unused-var-strict]: unused variable l.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind: feature New user-facing feature.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants