Coder Social home page Coder Social logo

Comments (8)

DannyvdSluijs avatar DannyvdSluijs commented on June 1, 2024

Thanks for you input @alexkuc given you looking for answers I guess making the PR will be difficult. I’ll see when I have time available to sort it. Any specific question that is blocking you now?

from jsonmapper.

alexkuc avatar alexkuc commented on June 1, 2024

@DannyvdSluijs thank you for the fast response. I am currently going over source code of the existing middleware. I'll let you know if I get stuck with something. Thank you again.

from jsonmapper.

alexkuc avatar alexkuc commented on June 1, 2024

@DannyvdSluijs I managed to put something myself together. I think I got the general gist of the approach. Would you be open to PR to update docs?

from jsonmapper.

DannyvdSluijs avatar DannyvdSluijs commented on June 1, 2024

PR are more than welcome. That is the power of open source.

from jsonmapper.

alexkuc avatar alexkuc commented on June 1, 2024

@DannyvdSluijs I was about to start writing new README page but I just realised something so wanted to double check with you:

public function push(callable $middleware, string $name = ''): self;

As far as I can tell JsonMapperInterface does not really enforce middleware type, it's purely by convention that middleware should use MiddlewareInterface and MiddlewareLogicInterface?

from jsonmapper.

DannyvdSluijs avatar DannyvdSluijs commented on June 1, 2024

Hi @alexkuc

I've been busy so you'll have to excuse me for my late reply.
Your assumptions are correct. This has been an oversight which i want to address at some point but since that would impact others as well that should only be done in a new mayor release.

from jsonmapper.

DannyvdSluijs avatar DannyvdSluijs commented on June 1, 2024

@alexkuc I wanted to check with you if you still need any help, either in understanding JsonMapper or in opening a PR?

from jsonmapper.

alexkuc avatar alexkuc commented on June 1, 2024

@DannyvdSluijs all is good

from jsonmapper.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.