Skip to content
This repository has been archived by the owner on Mar 4, 2020. It is now read-only.

feat(Text): add prop interfaces #154

Merged
merged 4 commits into from Aug 31, 2018
Merged

Conversation

kuzhelov
Copy link
Contributor

This PR introduces props interface for Text component, as part of #117

timestamp: boolean
truncated: boolean
styles: IComponentPartStylesInput
variables: ComponentVariablesInput
Copy link
Collaborator

Choose a reason for hiding this comment

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

@kuzhelov shouldn't all these be optional?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yes, exactly - updating..

@codecov
Copy link

codecov bot commented Aug 30, 2018

Codecov Report

Merging #154 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #154   +/-   ##
=======================================
  Coverage   68.02%   68.02%           
=======================================
  Files         101      101           
  Lines        1370     1370           
  Branches      260      269    +9     
=======================================
  Hits          932      932           
  Misses        436      436           
  Partials        2        2
Impacted Files Coverage Δ
src/themes/teams/components/Text/textStyles.ts 11.11% <ø> (ø) ⬆️
src/components/Text/Text.tsx 100% <100%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e5ef249...41e1cfc. Read the comment docs.

@kuzhelov kuzhelov merged commit a5814f8 into master Aug 31, 2018
@kuzhelov kuzhelov deleted the feat/add-prop-interfaces-for-text branch August 31, 2018 19:35
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants