Skip to content

Conversation

@hanoak20
Copy link
Contributor

No description provided.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We need to overlook the type of Migration. As of now its ok.

Copy link
Contributor

@himanshu-contentstack himanshu-contentstack left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please look into the comments and do the needful

owner: req?.body?.token_payload?.user_id,
});

project.migration[0].name = req?.body?.name;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Use destructure operator instead of doing req.body.... everytime

_id: projectId,
"migration._id": migrationId,
org_id: orgId,
region: req?.body?.token_payload?.region,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Use destructure operator

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we need that, wouldn't that be an extra?

Copy link
Contributor

@himanshu-contentstack himanshu-contentstack left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Schema structure needs to be overlooked

@himanshu-contentstack himanshu-contentstack merged commit f36c0e7 into dev Jan 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants