Open LuizAsFight opened 3 months ago
Hey @LuizAsFight - is there any movement on this one?
@petertonysmith94 I am asking for some opinions, we should move this soon.
What you think of the idea? I changed the supportedMethods
prop to be unsupportedMethods
, the reasoning is that normally connectors will support most of the methods, so I am doing the dev inform only the exception (methods not supported)
@LuizAsFight I can share my opinions :)
The addition of unsupportedMethods
to the connectors, seems reasonable to me.
I get the rational behind the isMethodSupported
and getSupportedConnectors
methods. Just wondering whether they need to go onto the Fuel
SDK, or could they sit externally as helper functions?
The final step 4 could easily be added to the callMethod
function.
this is a proposal is for improving the spec below https://github.com/FuelLabs/fuel-connectors/wiki
step 1 - add a new section:
Supported Methods
Include on
FuelConnectors
interface a way to provide the methods that this connector supportsThis enables improvements on UX for the Connectors UI by:
Allowing UI to hide the connectors that don't support the method that the user wants to call
step 2 - include in Connector method list
step 3 - add to Fuel SDK method list
supportedMethods
)step 4 (optional) - create a way to block methods that are not supported when they are called
This is open for discussion. One option is to add overrides for each connector function, executing first the
isMethodSupported
For this we would need to add to Fuel SDK method list overriding methods from Fuel Connector