Skip to content

feat: use /v1 namespace for API#7

Merged
Bnyro merged 1 commit intolibre-tube:mainfrom
FineFindus:feat/api-namespace
Apr 29, 2026
Merged

feat: use /v1 namespace for API#7
Bnyro merged 1 commit intolibre-tube:mainfrom
FineFindus:feat/api-namespace

Conversation

@FineFindus
Copy link
Copy Markdown
Contributor

Improves the semantics for a breaking changes, by making it easier to differentiate between /v1/call and /v2/call, instead of /call and /v2/call.

Copy link
Copy Markdown
Member

@Bnyro Bnyro left a comment

Choose a reason for hiding this comment

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

Yeah, definitely makes sense, as we probably will have breaking changes at some time.

I'm not motivated to rebase libre-tube/LibreTube#8367 currently (I'd like to keep this functional for people to test it), so I'll merge it when I work on that PR next time.

Thanks 👍

@Bnyro Bnyro force-pushed the feat/api-namespace branch from cfb69bf to 965e6b8 Compare April 29, 2026 18:03
Improves the semantics for a breaking changes, by making it easier to
differentiate between `/v1/call` and `/v2/call`, instead of `/call` and
`/v2/call`.
@Bnyro Bnyro force-pushed the feat/api-namespace branch from 965e6b8 to 4aae2a7 Compare April 29, 2026 18:04
@Bnyro
Copy link
Copy Markdown
Member

Bnyro commented Apr 29, 2026

Fyi, dotenvy was removed in #9, so I dropped your commit related to it.

@Bnyro Bnyro merged commit e9dd426 into libre-tube:main Apr 29, 2026
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.

2 participants