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.

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

Replace example issue #17

Merged
merged 5 commits into from Nov 9, 2016

Conversation

Projects
None yet
2 participants
@acka47
Contributor

acka47 commented Nov 9, 2016

Fixes #9.

I might have gotten a bit tangled up with this, however, now we have an issue with

  • a reference to a commit
  • a mention
  • a reference to another issue
  • a reference from another issue
  • a reference from a commit

I will try to resist the urge replacing it once again when the reference from this PR is added...

acka47 added some commits Nov 9, 2016

Replace GitHub issue example screenshot.
Fixes #9.

 # Bitte geben Sie eine Commit-Beschreibung für Ihre Änderungen ein. Zeilen,
Replaced issue screenshot for the last time.
Now with additional commit reference.

@acka47 acka47 added the review label Nov 9, 2016

Replace example issue screenshot for the last time.
This time with reference by pull request.
Update screenshots for consistency (#9)
Full window with drop shadow for both screenshots
@fsteeg

This comment has been minimized.

Show comment
Hide comment
@fsteeg

fsteeg Nov 9, 2016

Collaborator

Updated screenshots for consistency (full window with drop shadow for both screenshots).

Collaborator

fsteeg commented Nov 9, 2016

Updated screenshots for consistency (full window with drop shadow for both screenshots).

@fsteeg fsteeg assigned acka47 and unassigned fsteeg Nov 9, 2016

@acka47

This comment has been minimized.

Show comment
Hide comment
@acka47

acka47 Nov 9, 2016

Contributor

Hmm, now the link from the pull request is lost from the screenshot... However, as we introduce PRs later anyway it might not be that important...

Contributor

acka47 commented Nov 9, 2016

Hmm, now the link from the pull request is lost from the screenshot... However, as we introduce PRs later anyway it might not be that important...

@acka47

This comment has been minimized.

Show comment
Hide comment
@acka47

acka47 Nov 9, 2016

Contributor

I now also adjusted the links. Please merge if you are ok with it.

Contributor

acka47 commented Nov 9, 2016

I now also adjusted the links. Please merge if you are ok with it.

@acka47 acka47 assigned fsteeg and unassigned acka47 Nov 9, 2016

@fsteeg fsteeg merged commit c847964 into master Nov 9, 2016

@fsteeg fsteeg deleted the 9-exampleIssue branch Nov 9, 2016

@fsteeg fsteeg removed the review label Nov 9, 2016

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