Skip to content

PR by hatboysam-test#13

Open
alicethecoder wants to merge 1 commit into
mainfrom
hatboysam-test-patch-1
Open

PR by hatboysam-test#13
alicethecoder wants to merge 1 commit into
mainfrom
hatboysam-test-patch-1

Conversation

@alicethecoder

@alicethecoder alicethecoder commented Aug 30, 2021

Copy link
Copy Markdown
Collaborator

Quick description

@codeapprove

codeapprove Bot commented Aug 30, 2021

Copy link
Copy Markdown

👋 Review this Pull Request on CodeApprove here:
https://codeapprove.com/pr/codeapprove/meta/13

@alicethecoder alicethecoder changed the title Update index.js PR by hatboysam-test Aug 30, 2021
@codeapprove-dev

Copy link
Copy Markdown

👋 Review this Pull Request on CodeApprove here:
http://localhost:8080/pr/codeapprove/meta/13

@hatboysam hatboysam left a comment

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.

Automated comment from CodeApprove (go to review)

✔️ Approved

This looks good 👍


👀 @hatboysam-test it's your turn please take a look

@hatboysam hatboysam left a comment

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.

Automated comment from CodeApprove (go to review)

Comment


👀 @hatboysam-test it's your turn please take a look

@hatboysam hatboysam left a comment

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.

Automated comment from CodeApprove (go to review)

❌ Undo Approval

Undo approval

@hatboysam hatboysam left a comment

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.

Automated comment from CodeApprove (go to review)

Comment

@hatboysam hatboysam left a comment

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.

Automated comment from CodeApprove (go to review)

Add comment

@hatboysam hatboysam left a comment

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.

Automated comment from CodeApprove (go to review)

Add a comment

@hatboysam hatboysam left a comment

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.

Automated comment from CodeApprove (go to review)

Comment

@hatboysam hatboysam left a comment

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.

Automated comment from CodeApprove (go to review)

Again

@hatboysam hatboysam left a comment

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.

Automated comment from CodeApprove (go to review)

OK one more try

@hatboysam hatboysam left a comment

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.

Automated comment from CodeApprove (go to review)

Again again

@hatboysam hatboysam left a comment

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.

Automated comment from CodeApprove (go to review)

✔️ Approved

I approve


👀 @hatboysam-test it's your turn please take a look

@hatboysam hatboysam left a comment

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.

Automated comment from CodeApprove (go to review)

Ok


👀 @hatboysam-test it's your turn please take a look

@hatboysam hatboysam left a comment

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.

Automated comment from CodeApprove (go to review)

Comment


👀 @hatboysam-test it's your turn please take a look

@hatboysam hatboysam left a comment

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.

Automated comment from CodeApprove (go to review)

✔️ Approved (1 unresolved comments)


In: Discussion
Ok


👀 @hatboysam-test it's your turn please take a look

@hatboysam hatboysam left a comment

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.

Automated comment from CodeApprove (go to review)

✔️ Approved (2 unresolved comments)


In: Discussion
Draft


👀 @hatboysam-test it's your turn please take a look

@hatboysam hatboysam left a comment

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.

Automated comment from CodeApprove (go to review)

In: Discussion
Resolved


In: Discussion
Resolved


👀 @hatboysam-test it's your turn please take a look

@hatboysam hatboysam left a comment

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.

Automated comment from CodeApprove (go to review)

✔️ Approved (1 unresolved comments)


In: Discussion
OK


👀 @hatboysam-test it's your turn please take a look

@hatboysam hatboysam left a comment

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.

Automated comment from CodeApprove (go to review)

✔️ Approved (2 unresolved comments)


In: Discussion
Ok


👀 @hatboysam-test it's your turn please take a look

@hatboysam hatboysam left a comment

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.

Automated comment from CodeApprove (go to review)

❌ Undo Approval (2 unresolved comments)

Undo approval


👀 @hatboysam-test it's your turn please take a look

@hatboysam hatboysam left a comment

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.

Automated comment from CodeApprove (go to review)

✔️ Approved (3 unresolved comments)


👀 @hatboysam-test it's your turn please take a look

@hatboysam hatboysam left a comment

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.

Automated comment from CodeApprove (go to review)

❌ Undo Approval (3 unresolved comments)

Undo approval


👀 @hatboysam-test it's your turn please take a look

@hatboysam hatboysam left a comment

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.

Automated comment from CodeApprove (go to review)

✔️ Approved (4 unresolved comments)


👀 @hatboysam-test it's your turn please take a look

@hatboysam hatboysam left a comment

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.

Automated comment from CodeApprove (go to review)

❌ Undo Approval (5 unresolved comments)


👀 @hatboysam-test it's your turn please take a look

@hatboysam hatboysam left a comment

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.

Automated comment from CodeApprove (go to review)

✔️ Approved (5 unresolved comments)

Undo approval


👀 @hatboysam-test it's your turn please take a look

@hatboysam hatboysam left a comment

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.

Automated comment from CodeApprove (go to review)

❌ Undo Approval (5 unresolved comments)

This is a comment with undo approval


👀 @hatboysam-test it's your turn please take a look

@hatboysam hatboysam left a comment

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.

Automated comment from CodeApprove (go to review)

✔️ Approved (5 unresolved comments)

Approve with a comment


👀 @hatboysam-test it's your turn please take a look

@hatboysam hatboysam left a comment

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.

Automated comment from CodeApprove (go to review)

❌ Undo Approval (5 unresolved comments)

Undo with a comment


👀 @hatboysam-test it's your turn please take a look

@hatboysam hatboysam left a comment

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.

Automated comment from CodeApprove (go to review)

Just a comment


👀 @hatboysam-test it's your turn please take a look

@hatboysam hatboysam left a comment

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.

Automated comment from CodeApprove (go to review)

✔️ Approved (5 unresolved comments)

Approve with a comment


👀 @hatboysam-test it's your turn please take a look

@hatboysam hatboysam left a comment

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.

Automated comment from CodeApprove (go to review)

✔️ Approved (5 unresolved comments)

Approve this


👀 @hatboysam-test it's your turn please take a look

@hatboysam hatboysam left a comment

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.

Automated comment from CodeApprove (go to review)

✔️ Approved (2 unresolved comments)


In: Discussion
Resolve


In: Discussion
Resolve


👀 @hatboysam-test it's your turn please take a look

@hatboysam hatboysam left a comment

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.

Automated comment from CodeApprove (go to review)

✔️ Approved (1 unresolved comments)

Resolving discussions


👀 @hatboysam-test it's your turn please take a look

@hatboysam hatboysam left a comment

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.

Automated comment from CodeApprove (go to review)

In: Discussion
Resolve


👀 @hatboysam-test it's your turn please take a look

@hatboysam hatboysam left a comment

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.

Automated comment from CodeApprove (go to review)

Unresolved!


👀 @hatboysam-test it's your turn please take a look

@hatboysam hatboysam dismissed their stale review May 28, 2022 18:32

Reviewer removed on CodeApprove

@hatboysam hatboysam left a comment

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.

Automated comment from CodeApprove (go to review)

✔️ Approved (1 unresolved comments)

New approval


👀 @hatboysam-test it's your turn please take a look

@hatboysam hatboysam left a comment

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.

Automated comment from CodeApprove (go to review)

Keeping approval


In: Discussion
Resolved


👀 @hatboysam-test it's your turn please take a look

@hatboysam hatboysam left a comment

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.

Seeing if comment after approval rmoves it

@hatboysam hatboysam left a comment

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.

Automated comment from CodeApprove (go to review)

✔️ Approved (1 unresolved comments)

Justnone comment


In: index.js:

> Line 6
}

Refactor


👀 @hatboysam-test it's your turn please take a look

@hatboysam hatboysam left a comment

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.

Automated comment from CodeApprove (go to review)

In: index.js:

> Line 6
}

Resolved


👀 @hatboysam-test it's your turn please take a look

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants