search

metacall / faas

Reimplementation of MetaCall FaaS platform written in TypeScript.
https://dashboard.metacall.io
Apache License 2.0
13 stars 20 forks source link

Refractor the existing controllers #30

Open HeeManSu opened 4 months ago

HeeManSu commented 4 months ago

The TODO task has been added in several places. Complete them and refactor the other controllers for any potential improvements.

Screenshot from 2024-02-29 23-21-01 Screenshot from 2024-02-29 23-21-23

khareyash05 commented 4 months ago

On it! Kindly assign me this

viferga commented 1 month ago

This is almost done but there's something remaining, all the lines containing this:

// TODO: Isn't this available inside protocol package? We MUST reuse it

This may be common and we either have to find one existing type/interface in the protocol or export the one in the faas into the protocol.

Maybe we can close this issue and create a new one only for the interface thing. What do you think @HeeManSu ?