Skip to content
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.

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

Isomorphic Request Body Links #2492

Closed
wants to merge 8 commits into from
Closed

Conversation

luckybroman5
Copy link

@luckybroman5 luckybroman5 commented Mar 5, 2021

Feedback from TSC Meeting:

  • Rename "Fragments", suggestions: Mappings, Expressions, etc..
  • Reusability, or the ability to add into Components
  • Initial proposal is somewhat too complex. Its usage should be simpler.
  • Representing Form Data
  • More thought Around the OperationRef Portions.
  • Ability to ensure tooling can "Validate" these links against Schemas in both Requests and Responses. This might not be feasible with runtime expressions

@luckybroman5 luckybroman5 changed the title Backlinks and Allowing for Links to Reference portions of the linked Request Body Isomorphic Request Body Links Mar 5, 2021
@luckybroman5
Copy link
Author

Update Regarding Progress here:

I am still working on a better pass before asking for another review. I'm in the process of developing some "test" tooling to see what makes the most sense, while also going for what is the most concise.

@MikeRalphson
Copy link
Member

As a result of #2604 the filename for this proposal should change to use a dated prefix. Please could you update this PR to match? You can use the original PR date. Apologies for the extra work.

@lornajane
Copy link
Contributor

This pull request has been open for a long time without any activity. It isn't ready to discuss as-is, so it will be closed. As always, proposals are welcome, so if this still seems relevant, let's re-start the discussion and look at how recent changes such as the Workflows SIG may change some of the ideas.

@lornajane lornajane closed this Jan 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants