Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Test #2

Closed
wants to merge 3 commits into from
Closed

Test #2

wants to merge 3 commits into from

Conversation

podium-mu
Copy link
Collaborator

Testing integrations

@CJPoll CJPoll requested review from CJPoll and removed request for CJPoll October 24, 2017 02:53
CJPoll
CJPoll reviewed Oct 24, 2017
View reviewed changes
Copy link
Owner

@CJPoll CJPoll left a comment

Choose a reason for hiding this comment

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

This is the body

CJPoll
CJPoll requested changes Oct 24, 2017
View reviewed changes
Copy link
Owner

@CJPoll CJPoll left a comment

Choose a reason for hiding this comment

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

Requesting changes arbitrarily

CJPoll
CJPoll approved these changes Oct 24, 2017
View reviewed changes
Copy link
Owner

@CJPoll CJPoll left a comment

Choose a reason for hiding this comment

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

Approved?

CJPoll
CJPoll requested changes Oct 24, 2017
View reviewed changes
Copy link
Owner

@CJPoll CJPoll left a comment

Choose a reason for hiding this comment

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

asdfasdf

CJPoll
CJPoll approved these changes Oct 24, 2017
View reviewed changes
Copy link
Owner

@CJPoll CJPoll left a comment

Choose a reason for hiding this comment

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

asdfasdf

CJPoll
CJPoll requested changes Oct 24, 2017
View reviewed changes
Copy link
Owner

@CJPoll CJPoll left a comment

Choose a reason for hiding this comment

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

asdfasdf

CJPoll
CJPoll approved these changes Oct 24, 2017
View reviewed changes
Copy link
Owner

@CJPoll CJPoll left a comment

Choose a reason for hiding this comment

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

asdfadsfa

CJPoll
CJPoll requested changes Oct 24, 2017
View reviewed changes
Copy link
Owner

@CJPoll CJPoll left a comment

Choose a reason for hiding this comment

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

asdfasdf

CJPoll
CJPoll approved these changes Oct 24, 2017
View reviewed changes
Copy link
Owner

@CJPoll CJPoll left a comment

Choose a reason for hiding this comment

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

asdfasdf

CJPoll
CJPoll requested changes Oct 24, 2017
View reviewed changes
Copy link
Owner

@CJPoll CJPoll left a comment

Choose a reason for hiding this comment

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

asdfasdf

CJPoll
CJPoll approved these changes Oct 24, 2017
View reviewed changes
Copy link
Owner

@CJPoll CJPoll left a comment

Choose a reason for hiding this comment

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

asdfasdf

CJPoll
CJPoll requested changes Oct 24, 2017
View reviewed changes
Copy link
Owner

@CJPoll CJPoll left a comment

Choose a reason for hiding this comment

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

asdfasdfafd

CJPoll
CJPoll approved these changes Oct 24, 2017
View reviewed changes
Copy link
Owner

@CJPoll CJPoll left a comment

Choose a reason for hiding this comment

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

This is gold!

CJPoll
CJPoll requested changes Oct 24, 2017
View reviewed changes
Copy link
Owner

@CJPoll CJPoll left a comment

Choose a reason for hiding this comment

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

This is 💩

CJPoll
CJPoll reviewed Oct 24, 2017
View reviewed changes
Copy link
Owner

@CJPoll CJPoll left a comment

Choose a reason for hiding this comment

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

This is a comment

CJPoll
CJPoll reviewed Oct 24, 2017
View reviewed changes
Copy link
Owner

@CJPoll CJPoll left a comment

Choose a reason for hiding this comment

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

This is a comment

CJPoll
CJPoll reviewed Oct 24, 2017
View reviewed changes
assets/css/app.css
{
box-sizing: border-box;
}
Copy link
Owner

Choose a reason for hiding this comment

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

Please no

@CJPoll CJPoll self-requested a review October 24, 2017 03:41
CJPoll
CJPoll approved these changes Oct 24, 2017
View reviewed changes
Copy link
Owner

@CJPoll CJPoll left a comment

Choose a reason for hiding this comment

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

Approve

CJPoll
CJPoll requested changes Oct 24, 2017
View reviewed changes
Copy link
Owner

@CJPoll CJPoll left a comment

Choose a reason for hiding this comment

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

This is

a multi-line

rejection

CJPoll
CJPoll approved these changes Oct 24, 2017
View reviewed changes
Copy link
Owner

@CJPoll CJPoll left a comment

Choose a reason for hiding this comment

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

multi

line

approval

CJPoll
CJPoll requested changes Oct 24, 2017
View reviewed changes
Copy link
Owner

@CJPoll CJPoll left a comment

Choose a reason for hiding this comment

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

ulti
line

reject

CJPoll
CJPoll approved these changes Oct 24, 2017
View reviewed changes
Copy link
Owner

@CJPoll CJPoll left a comment

Choose a reason for hiding this comment

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

multi
line

aprprove

CJPoll
CJPoll requested changes Oct 24, 2017
View reviewed changes
Copy link
Owner

@CJPoll CJPoll left a comment

Choose a reason for hiding this comment

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

multi

line

rject

CJPoll
CJPoll approved these changes Oct 24, 2017
View reviewed changes
Copy link
Owner

@CJPoll CJPoll left a comment

Choose a reason for hiding this comment

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

multi

line

approve

CJPoll
CJPoll requested changes Oct 24, 2017
View reviewed changes
Copy link
Owner

@CJPoll CJPoll left a comment

Choose a reason for hiding this comment

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

single-line reject

CJPoll
CJPoll approved these changes Oct 24, 2017
View reviewed changes
Copy link
Owner

@CJPoll CJPoll left a comment

Choose a reason for hiding this comment

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

this is
approval

CJPoll
CJPoll requested changes Oct 24, 2017
View reviewed changes
Copy link
Owner

@CJPoll CJPoll left a comment

Choose a reason for hiding this comment

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

This is
reject

CJPoll
CJPoll approved these changes Oct 25, 2017
View reviewed changes
Copy link
Owner

@CJPoll CJPoll left a comment

Choose a reason for hiding this comment

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

This is approved

CJPoll
CJPoll requested changes Oct 25, 2017
View reviewed changes
Copy link
Owner

@CJPoll CJPoll left a comment

Choose a reason for hiding this comment

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

reject

CJPoll
CJPoll approved these changes Oct 25, 2017
View reviewed changes
Copy link
Owner

@CJPoll CJPoll left a comment

Choose a reason for hiding this comment

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

Approved!

CJPoll
CJPoll requested changes Oct 25, 2017
View reviewed changes
Copy link
Owner

@CJPoll CJPoll left a comment

Choose a reason for hiding this comment

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

This sucks

@CJPoll CJPoll closed this Oct 25, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@CJPoll CJPoll CJPoll requested changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@podium-mu @CJPoll