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(components): migrate Text to TypeScript #581

Merged
merged 4 commits into from
May 4, 2020

Conversation

connor-baer
Copy link
Member

Addresses #516.

Purpose

I was testing #580 using the Text component.

Approach and changes

  • create custom styled instance to type the theme as described in the Emotion docs
  • migrate Text component to TypeScript
  • remove static properties (Text.KILO, Text.MEGA, Text.GIGA) since they're no longer needed with TypeScript and I couldn't figure out how to add types for them

Definition of done

  • Development completed
  • Reviewers assigned
  • Unit and integration tests
  • Meets minimum browser support
  • Meets accessibility requirements

@connor-baer connor-baer added the feature A new feature or enhancement label May 1, 2020
@vercel
Copy link

vercel bot commented May 1, 2020

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/sumup-oss/circuit-ui/ho8s60jpz
✅ Preview: https://circuit-ui-git-feature-typescript-text.sumup-oss.now.sh

@codecov
Copy link

codecov bot commented May 1, 2020

Codecov Report

Merging #581 into feature/typescript-jest will decrease coverage by 0.04%.
The diff coverage is 100.00%.

Impacted file tree graph

@@                     Coverage Diff                     @@
##           feature/typescript-jest     #581      +/-   ##
===========================================================
- Coverage                    93.42%   93.37%   -0.05%     
===========================================================
  Files                          151      152       +1     
  Lines                         2433     2431       -2     
  Branches                       682      683       +1     
===========================================================
- Hits                          2273     2270       -3     
- Misses                         132      133       +1     
  Partials                        28       28              
Impacted Files Coverage Δ
src/components/Blockquote/Blockquote.js 100.00% <100.00%> (ø)
...nents/Button/components/PlainButton/PlainButton.js 100.00% <100.00%> (ø)
src/components/Text/Text.tsx 100.00% <100.00%> (ø)
src/components/Toggle/Toggle.js 100.00% <100.00%> (ø)
src/styles/styled.ts 100.00% <100.00%> (ø)
...s/Button/components/RegularButton/RegularButton.js 94.28% <0.00%> (-2.86%) ⬇️

connor-baer and others added 4 commits May 4, 2020 14:26
* test(configs): fix Jest config for TypeScript

feature/typescript-jest

feature/typescript-jest

* docs: update test command

feature/typescript-jest
feature/typescript-text

feature/typescript-text
@github-actions
Copy link
Contributor

github-actions bot commented May 4, 2020

🎉 This PR is included in version 2.0.0-alpha.13 🎉

The release is available on:

Your semantic-release bot 📦🚀

connor-baer added a commit that referenced this pull request May 4, 2020
@connor-baer connor-baer deleted the feature/typescript-text branch May 5, 2020 09:04
@github-actions
Copy link
Contributor

github-actions bot commented Jun 8, 2020

🎉 This PR is included in version 2.0.0-beta.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

@herberthenrique herberthenrique mentioned this pull request Jun 9, 2020
49 tasks
@github-actions
Copy link
Contributor

🎉 This PR is included in version 2.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Successfully merging this pull request may close these issues.

2 participants