Skip to content

WIP: [API] Update URL format for Test Lexbox#369

Draft
papeh wants to merge 1 commit intomasterfrom
fix/login
Draft

WIP: [API] Update URL format for Test Lexbox#369
papeh wants to merge 1 commit intomasterfrom
fix/login

Conversation

@papeh
Copy link
Copy Markdown
Contributor

@papeh papeh commented Apr 28, 2026

Before, production users could log in and list projects by API: https://admin.languagedepot.org/api/user/{Username}/projects and test users by
https://admin-qa.languagedepot.org/api/user/{Username}/projects Now, the 'admin' prefix is obsolete and available only in production. I assume but have not verified that production users can use https://languagedepot.org/api/user/{Username}/projects and testers
https://staging.languagedepot.org/api/user/{Username}/projects

Trim the leading '.' or '-' needed for prepending remaining subdomains in API calls.


This change is Reviewable

Before, production users could log in and list projects by API:
https://admin.languagedepot.org/api/user/{Username}/projects
and test users by
https://admin-qa.languagedepot.org/api/user/{Username}/projects
Now, the 'admin' prefix is obsolete and available only in production.
I assume but have not verified that production users can use
https://languagedepot.org/api/user/{Username}/projects
and testers
https://staging.languagedepot.org/api/user/{Username}/projects

Trim the leading '.' or '-' needed for prepending remaining subdomains in API calls.
@papeh
Copy link
Copy Markdown
Contributor Author

papeh commented Apr 28, 2026

This is technically a breaking change, but we don't expect it to break anything that is currently working (unless someone has a self-hosted Language Depot). Does it need a +semver:major?

@github-actions
Copy link
Copy Markdown

Test Results

       8 files  ±0     333 suites  ±0   2h 18m 20s ⏱️ - 4m 41s
   988 tests ±0     932 ✔️ ±0    56 💤 ±0  0 ±0 
3 145 runs  ±0  3 022 ✔️ ±0  123 💤 ±0  0 ±0 

Results for commit 19b8377. ± Comparison against base commit 1ab24b9.

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.

1 participant