Skip to content

3.1: sync Appendix E from 3.2 - #4879

Merged
ralfhandl merged 1 commit into
OAI:v3.1-devfrom
ralfhandl:3.1-sync-appendix-E-from-3.2
Sep 2, 2025
Merged

3.1: sync Appendix E from 3.2#4879
ralfhandl merged 1 commit into
OAI:v3.1-devfrom
ralfhandl:3.1-sync-appendix-E-from-3.2

Conversation

@ralfhandl

@ralfhandlralfhandl commented Aug 17, 2025•
edited
Loading

Copy link
Copy Markdown
Contributor

This syncs useful explanations and caveats from v3.2, Appendix E.

  • 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

@ralfhandl
ralfhandl requested review from a team as code ownersAugust 17, 2025 12:19
@ralfhandlralfhandl added the approved pr port PRs that just port an approved PR to another version label Aug 17, 2025
@ralfhandlralfhandl added this to the v3.1.2 milestone Aug 17, 2025
@ralfhandlralfhandl linked an issue Aug 17, 2025 that may be closed by this pull request
v3.1: sync percent-encoding improvements from 3.2? #4878
Closed
@ralfhandlralfhandl mentioned this pull request Aug 17, 2025
Closed
@handrews

handrews commented Aug 17, 2025

Copy link
Copy Markdown
Member

@ralfhandl oh you meant this set of Appendix E changes rather than the more recent set! Sorry about closing the issue- I assumed you meant the Appendix E changes that had just been merged in the last week. I'd forgotten about this change.

lornajane
lornajane approved these changes Aug 17, 2025
View reviewed changes

@lornajanelornajane left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks!

@lornajane
lornajane requested a review from a teamAugust 17, 2025 15:34
@ralfhandlralfhandl mentioned this pull request Aug 18, 2025
Closed
@ralfhandl

ralfhandl commented Aug 21, 2025

Copy link
Copy Markdown
ContributorAuthor

@OAI/tsc Second TSC review, please!

mikekistler
mikekistler approved these changes Sep 2, 2025
View reviewed changes

@mikekistlermikekistler left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Looks good. 👍

@ralfhandl
ralfhandl merged commit c7a7629 into OAI:v3.1-devSep 2, 2025
2 checks passed
@ralfhandl
ralfhandl deleted the 3.1-sync-appendix-E-from-3.2 branch September 2, 2025 17:17
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@lornajanelornajanelornajane approved these changes
@handrewshandrewshandrews approved these changes
@mikekistlermikekistlermikekistler approved these changes
@miquimiquiAwaiting requested review from miqui

Assignees

No one assigned

Labels

approved pr portPRs that just port an approved PR to another version

Projects

None yet

Milestone

v3.1.2

Development

Successfully merging this pull request may close these issues.

v3.1: sync percent-encoding improvements from 3.2?

4 participants

@ralfhandl@handrews@lornajane@mikekistler