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.

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

feat(Comment): add component #392

Closed
wants to merge 3 commits into from
Closed

feat(Comment): add component #392

wants to merge 3 commits into from

Conversation

eanplatter
Copy link
Contributor

@eanplatter eanplatter commented Aug 13, 2016

WIP

Fixes #187

@levithomason levithomason changed the title Add Card Component to v1 API Update Comment Component to v1 API Aug 13, 2016
@levithomason levithomason changed the title Update Comment Component to v1 API Add Comment Component Aug 13, 2016
@codecov-io
Copy link

Current coverage is 92.33% (diff: 63.82%)

Merging #392 into master will decrease coverage by 2.53%

@@             master       #392   diff @@
==========================================
  Files            85         81     -4   
  Lines          1111        991   -120   
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
- Hits           1054        915   -139   
- Misses           57         76    +19   
  Partials          0          0          

Powered by Codecov. Last update 9c8c668...565e2c6

@levithomason levithomason changed the title Add Comment Component Comment: add component Aug 18, 2016
@levithomason levithomason changed the title Comment: add component feat(Comment): add component Sep 20, 2016
<Comment.Actions>
<a class="reply">Reply</a>
</Comment.Actions>
</Comment.Conent>
Copy link
Contributor

@typehorror typehorror Sep 28, 2016

Choose a reason for hiding this comment

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

</Comment.Content>

@levithomason
Copy link
Member

This branch needs updated to keep from getting to far behind the latest updates.

@layershifter
Copy link
Member

Looks like dead PR, I can start it from scratch ⚡

@layershifter
Copy link
Member

Replaced with #584.

@levithomason
Copy link
Member

@eanplatter !!!!!!!!!!!!!!!!!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants