Skip to content
New issue

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

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

Already on GitHub? # to your account

[cds^9] cds.middlewares.after #968

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

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?

# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants