add NCW feature

[amper-fb]

Pull Request Description

Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.

Fixes (link to the issue here)

Type of change

Please delete options that are not relevant.

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Chore / Documentation
• Breaking change (fix or feature that would cause existing functionality to not work as expected)

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

• Locally tested against Fireblocks API

Checklist:

• I have performed a self-review of my own code
• I have made corresponding changes to the documentation
• Any dependent changes have been merged and published in downstream modules
• I have added corresponding labels to the PR

[YoavBZ]

Thanks @amper-fb for the contribution.
We haven't introduced namespaces in our SDK so far so adding one really changes the UX.
We might better do this in our new SDK and keep the current structure of this SDK.
@dsahni @zoharsf

[amper-fb]

Thanks @amper-fb for the contribution. We haven't introduced namespaces in our SDK so far so adding one really changes the UX. We might better do this in our new SDK and keep the current structure of this SDK. @dsahni @zoharsf

how does it "really change the UX"?

[YoavBZ]

Thanks @amper-fb for the contribution. We haven't introduced namespaces in our SDK so far so adding one really changes the UX. We might better do this in our new SDK and keep the current structure of this SDK. @dsahni @zoharsf

how does it "really change the UX"?

We talked about adding namespaces a long time ago and eventually decided not to do it in this SDK (we do that in our next-gen SDK)
Adding a namespace to only one group of methods is confusing and doesn't follow the current structure