fix: parameterize chainId

[sahar-fehri]

Explanation

Parameterize chainId in getNftInformationFromApi fct

References

Changelog

@metamask/assets-controllers

• FIXED: add networkClientId as argument in getNftInformationFromApi fct

Checklist

• I've updated the test suite for new or updated code as appropriate
• I've updated documentation (JSDoc, Markdown, etc.) for new or updated code as appropriate
• I've highlighted breaking changes using the "BREAKING" category above as appropriate
• I've prepared draft pull requests for clients and consumer packages to resolve any breaking changes

[mcmire]

Seems useful, but I'm not seeing any tests that confirm the new behavior. Can you add them?

[mcmire]

Nit: Since this is an optional argument what are your thoughts on placing this within an options bag? This would allow more optional arguments to be added in the future without worrying about passing them in a specific order.

Suggested change

	networkClientId?: NetworkClientId,
	{ networkClientId }: {
	networkClientId?: NetworkClientId,
	}