Skip to content

feat(request-headers): Document request headers #15172

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
nickvergessen merged 2 commits into from
Jun 6, 2025
Merged

feat(request-headers): Document request headers #15172

nickvergessen merged 2 commits into from
Jun 6, 2025

Conversation

@nickvergessen
Copy link
Member

@nickvergessen nickvergessen commented May 21, 2025 •
edited
Loading

☑️ Resolves

🏁 Checklist

  • ⛑️ Tests (unit and/or integration) are included or not possible
  • 📘 API documentation in docs/ has been updated or is not required
  • 🔖 Capability is added or not needed

@nickvergessen nickvergessen self-assigned this May 22, 2025
@nickvergessen nickvergessen added 2. developing feature: api 🛠️ OCS API for conversations, chats and participants labels May 22, 2025
@nickvergessen nickvergessen marked this pull request as draft May 22, 2025 06:31
@nickvergessen
Copy link
Member Author

nickvergessen commented Jun 2, 2025 •
edited
Loading

Optional things solved by nextcloud/openapi-extractor#250

@provokateurin
Copy link
Member

provokateurin commented Jun 3, 2025

Blocker solved by nextcloud/openapi-extractor#251.

Please try again and check if there are any remaining issues.

@nickvergessen nickvergessen force-pushed the techdebt/noid/request-headers branch from 6cf55ea to 8d40104 Compare June 6, 2025 08:34
@nickvergessen
Copy link
Member Author

nickvergessen commented Jun 6, 2025

Please try again and check if there are any remaining issues.

Tested against main and works now

@nickvergessen nickvergessen added this to the 🪺 Next Major (32) milestone Jun 6, 2025
@nickvergessen nickvergessen marked this pull request as ready for review June 6, 2025 08:35
@provokateurin
Copy link
Member

provokateurin commented Jun 6, 2025

v1.8.0 was tagged as well, so you can bump it

@nickvergessen nickvergessen merged commit 7919b07 into main Jun 6, 2025
82 checks passed
@nickvergessen nickvergessen deleted the techdebt/noid/request-headers branch June 6, 2025 09:14
@nickvergessen nickvergessen mentioned this pull request Jul 15, 2025
Merged
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@provokateurin provokateurin provokateurin approved these changes

@miaulalala miaulalala Awaiting requested review from miaulalala

Assignees

@nickvergessen nickvergessen

Labels

3. to review enhancement feature: api 🛠️ OCS API for conversations, chats and participants technical debt

Projects

None yet

Milestone

v22.0.0-beta.1

Development

Successfully merging this pull request may close these issues.

3 participants

@nickvergessen @provokateurin