Skip to content

Conversation

@sjvans
Copy link
Contributor

@sjvans sjvans commented May 29, 2024

Reverts #952 + adds more info

pr re cds.middlewares.before: #1094

@sjvans sjvans marked this pull request as draft May 29, 2024 10:19
@sjvans sjvans marked this pull request as ready for review May 29, 2024 10:26
@sjvans sjvans changed the title cds.middlewares.after [cds^8] cds.middlewares.after May 29, 2024
@renejeglinsky
Copy link
Contributor

We can merge cds8 PRs now. If this can also be merged, @sjvans please remove the do not merge label. Thx!

sjvans and others added 2 commits July 10, 2024 10:35
Co-authored-by: René Jeglinsky <rene.jeglinsky@sap.com>
@sjvans
Copy link
Contributor Author

sjvans commented Jul 10, 2024

@renejeglinsky i'll remove "do not merge" label once we've decided to make public

@sjvans sjvans changed the title [cds^8] cds.middlewares.after [cds^9] cds.middlewares.after Jan 9, 2025
@sjvans
Copy link
Contributor Author

sjvans commented Jan 9, 2025

@danjoa @johannes-vogel as our new protocol adapters pass the error into next (and we'll remove the old odata adapter in cds^9), i believe we could merge this one for the major... what do you think?

@sjvans sjvans changed the title [cds^9] cds.middlewares.after [cds^9] cds.middlewares.after + Base Protocol Adapter May 21, 2025
@renejeglinsky
Copy link
Contributor

@johannes-vogel could you give a review?

@renejeglinsky
Copy link
Contributor

@danjoa @johannes-vogel Ready for review

@sjvans sjvans changed the title [cds^9] cds.middlewares.after + Base Protocol Adapter [cds^9] cds.middlewares.after + PATCH vs. PUT vs. Replace + Base Protocol Adapter May 26, 2025
@sjvans sjvans changed the title [cds^9] cds.middlewares.after + PATCH vs. PUT vs. Replace + Base Protocol Adapter feat: cds.middlewares.after Aug 21, 2025
@sjvans sjvans marked this pull request as draft August 21, 2025 09:51
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.

3 participants