search

wingleung / remix-aws

AWS adapter for Remix
https://www.npmjs.com/package/remix-aws
MIT License
21 stars 6 forks source link

A boundary between the lambda handler and an adapter #2

Open kamtugeza opened 1 year ago

kamtugeza commented 1 year ago

It doesn't seem like an adapter will be changed on the fly, so why don't we go towards the "strategy" pattern and allow a user to decide what adapter to use, either custom or ready to use from the package?

P.S. Just a thought on how to implement the idea from the tread in #3173.

wingleung commented 1 year ago

I think this makes the createRequestHandler even more generic so we could raise an idea to keep a generic createRequestHandler in remix core and just make the adapters for each service according to their types 🤔