Refactor NodeManager constructor

[SanjoDeundiak]

Currently NodeManager::create function takes 10 arguments. It's time to move those arguments into into one or few structures

ockam/implementations/rust/ockam/ockam_api/src/nodes/service.rs

Lines 136 to 151 in 9294306

	impl NodeManager {
	/// Create a new NodeManager with the node name from the ockam CLI
	#[allow(clippy::too_many_arguments)]
	pub async fn create(
	ctx: &Context,
	node_name: String,
	node_dir: PathBuf,
	// Should be passed only when creating fresh node and we want it to get default root Identity
	identity_override: Option<IdentityOverride>,
	skip_defaults: bool,
	enable_credential_checks: bool,
	ac: Option<&AuthoritiesConfig>,
	project_id: Option<Vec<u8>>,
	api_transport: (TransportType, TransportMode, String),
	tcp_transport: TcpTransport,
	) -> Result<Self> {

---

We love helping new contributors!
If you have questions or need help as you work on your first Ockam contribution, please leave a comment on this discussion.
If you're looking for other issues to contribute to, checkout this discussion and labels - good first issue or help wanted

[vncsalencar]

Hi, @SanjoDeundiak. Do you think this is a good-first-issue candidate? I'd like to give this a go if so.

[mrinalwadhwa]

@vncsalencar I think so. Go ahead an give it a try, if you get stuck, please don't hesitate to ask questions, we're here to help.