Skip to content

v3.2: improve wording for servers object and url #4734

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
merged 1 commit into from
Jul 2, 2025
Merged

v3.2: improve wording for servers object and url #4734

merged 1 commit into from
Jul 2, 2025

Conversation

karenetheridge
Copy link
Member

I just implemented request matching with server urls, and found a few places where the spec could be made a little clearer.

  • schema changes are included in this pull request
  • schema changes are needed for this pull request but not done yet
  • no schema changes are needed for this pull request

@karenetheridge karenetheridge requested review from a team as code owners June 16, 2025 18:16
@karenetheridge karenetheridge mentioned this pull request Jun 16, 2025
Closed
handrews
handrews reviewed Jun 16, 2025
View reviewed changes
@handrews handrews changed the title improve wording for servers object and url v3.2: improve wording for servers object and url Jun 18, 2025
@ralfhandl ralfhandl requested a review from a team June 18, 2025 19:56
@handrews handrews mentioned this pull request Jun 19, 2025
Closed
mikekistler
mikekistler approved these changes Jul 2, 2025
View reviewed changes
Copy link
Contributor

@mikekistler mikekistler left a comment

Choose a reason for hiding this comment

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

Looks good. 👍

@handrews handrews merged commit 229e57a into OAI:v3.2-dev Jul 2, 2025
2 checks passed
@karenetheridge karenetheridge deleted the ether/server-url-default branch September 18, 2025 16:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@handrews handrews handrews left review comments

@ralfhandl ralfhandl ralfhandl approved these changes

@mikekistler mikekistler mikekistler approved these changes

+1 more reviewer

@philsturgeon philsturgeon philsturgeon approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@karenetheridge @philsturgeon @ralfhandl @handrews @mikekistler