Skip to content

Conversation

@joshsmith
Copy link
Contributor

@joshsmith joshsmith commented Oct 17, 2017

What's in this PR?

WIP. Adds beginning of GithubPullRequest model and icons, and other UI changes.

Edited before final merge

This PR

  • Adds a GithubPullRequest model
  • Adds a pull-request-icon component, which renders a closed/merged state
  • Renders the icon on the task-card component if the task is associated to a pull request

@joshsmith
Copy link
Contributor Author

This needs a test for the github/pull-request-icon component.

@joshsmith joshsmith force-pushed the add-github-pull-requests branch from 9178b23 to 6521f22 Compare October 18, 2017 05:45
Copy link
Contributor

@begedin begedin left a comment

Choose a reason for hiding this comment

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

task-card needs a test that it renders the pull request icon. I'll add it

- pull request icon renders when there's a pull request
- pull request icon does not render when there isn't a pull request
@begedin begedin merged commit ed0009f into develop Oct 18, 2017
@begedin begedin deleted the add-github-pull-requests branch October 18, 2017 08:05
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.

3 participants