timefold-quickstarts icon indicating copy to clipboard operation
timefold-quickstarts copied to clipboard

chore: align urls

Open TomCools opened this issue 9 months ago • 1 comments

DRAFT, DO NOT REVIEW YET, manual testing of UI ongoing.

Description of the change

Some URLs were not consistent,

  • Score analysis was a PUT call to analyse. This should be POST "score-analysis".
  • Starting a run should be POST to /<problem>, not POST to /<problem>/solve

Checklist

Development

  • [ ] The changes have been covered with tests, if necessary.
  • [ ] You have a green build, with the exception of the flaky tests.
  • [ ] UI and JS files are fully tested, the user interface works for all modules affected by your changes (e.g., solve and analyze buttons).
  • [ ] The network calls work for all modules affected by your changes (e.g., solving a problem).
  • [ ] The console messages are validated for all modules affected by your changes.

Code Review

  • [ ] This pull request includes an explanatory title and description.
  • [ ] The GitHub issue is linked.
  • [ ] At least one other engineer has approved the changes.
  • [ ] After PR is merged, inform the reporter.

TomCools avatar Feb 21 '25 08:02 TomCools

Still need to modify the REST Guide HTML

TomCools avatar Feb 25 '25 22:02 TomCools

This one got too old, don't see a lot of value of continuing this at this moment.

TomCools avatar Nov 10 '25 08:11 TomCools