Use project ID, not code, in CrdtMerge API

sillsdev / languageforge-lexbox

Lexbox, SIL linguistic data hub

MIT License

7 stars 2 forks source link

Use project ID, not code, in CrdtMerge API #1169

Closed rmunn closed 3 weeks ago

rmunn commented 3 weeks ago

Fix #1168.

We now use project code and ID in root folder name, use "crdt" and "fw" inside root folder no matter what the project's actual code or name is.

We also use project ID rather than code in the API request parameters. So now instead of POSTing to /sync/?projectCode=sena-3, you would POST to /sync/?projectId=0ebc5976-058d-4447-aaa7-297f8569f968.

github-actions[bot] commented 3 weeks ago

C# Unit Tests

75 tests ±0 75 :white_check_mark: ±0 5s :stopwatch: ±0s 13 suites ±0 0 :zzz: ±0 1 files ±0 0 :x: ±0

Results for commit cde1ab36. ± Comparison against base commit f72f3e09.

:recycle: This comment has been updated with latest results.

rmunn commented 3 weeks ago

Draft PR for now. Once https://github.com/sillsdev/languageforge-lexbox/pull/1170 finishes passing tests, I'll merge it, then rebase this PR on top of it and then we'll be ready for review.