Skip to content

Conversation

@sayalijoshi27
Copy link
Contributor

No description provided.

@sayalijoshi27 sayalijoshi27 requested a review from umeshmore45 May 9, 2024 13:35
<span className="AccountPage__circle AccountPage__circle_seven"></span>
</div>
<div className="AccountPage__action">
<div className="AccountPage__content">{props.children}</div>
Copy link
Contributor

Choose a reason for hiding this comment

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

add validation

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

const data: FieldMapType[] = []
schema?.forEach((field) => {
if (field?.ContentstackFieldType === "group") {
groupId = field.uid
Copy link
Contributor

Choose a reason for hiding this comment

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

add validation

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

obj?.child?.push(field)
}
else{
obj.child = [field]
Copy link
Contributor

Choose a reason for hiding this comment

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

add validation

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It is an assignment of an array so it can't be optional.

<ul>
{nestedList?.map((item: FieldMapType, index: number) => {
let outlineName = "";
if (item.uid) {
Copy link
Contributor

Choose a reason for hiding this comment

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

add validation

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

@AishDani AishDani merged commit 90700da into dev May 10, 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