Skip to content

log details of user#9

Open
jaya1001 wants to merge 2 commits intoHatsphere:masterfrom
jaya1001:master
Open

log details of user#9
jaya1001 wants to merge 2 commits intoHatsphere:masterfrom
jaya1001:master

Conversation

@jaya1001
Copy link
Copy Markdown
Contributor

No description provided.


for(i in n)
{
Map<item_id, Item> data = list[i].getData()
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Is this line valid

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Change it to

val data = list[i].getData()


for(i in n)
{
Map<item_id, Item> data = list[i].getData()
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Change it to

val data = list[i].getData()

Copy link
Copy Markdown
Contributor

@yashLadha yashLadha left a comment

Choose a reason for hiding this comment

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

Please see comments


val list: List<Product> = ArrayList<Product>()

CollectionReference itemRef = db.collection("Item")
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Change all datatypes to val

int order_id = item.order_id


CollectionReference productRef = db.collection("Product")
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Change data type to val

@yashLadha
Copy link
Copy Markdown
Contributor

Is it compiling and building? @jaya1001

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.

2 participants