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.

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

feat(Middleware): introduce InjectionMiddleware, with Table support first #927

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

feat(Middleware): introduce InjectionMiddleware, with Table support first #927

wants to merge 1 commit into from

Conversation

blizzz
Copy link
Member

@blizzz blizzz commented Mar 12, 2024

  • no more handcrafted Table instantiation and error handling on Controllers
  • switching to Attributes
  • again based on good Example of Talk

@blizzz
feat(Middleware): introduce InjectionMiddleware, with Table support f…
66f2d93 
…irst

- no more handcrafted Table instantiation and error handling on Controllers
- switching to Attributes

Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
@blizzz blizzz added enhancement New feature or request technical debt Technical issue 2. developing Work in progress labels Mar 12, 2024
@blizzz blizzz requested a review from juliusknorr March 12, 2024 21:24
@blizzz blizzz self-assigned this Mar 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@juliusknorr juliusknorr Awaiting requested review from juliusknorr

@enjeck enjeck Awaiting requested review from enjeck enjeck will be requested when the pull request is marked ready for review enjeck is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

@blizzz blizzz

Labels
2. developing Work in progress enhancement New feature or request technical debt Technical issue
Projects
📝 Office team
Status: 🏗️ In progress
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@blizzz