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(icons): added updated car icons #52

Conversation

hcafaq
Copy link
Contributor

@hcafaq hcafaq commented Aug 20, 2022

Pull Request

https://heycar.atlassian.net/jira/software/projects/HEYUI/boards/312?selectedIssue=HEYUI-141

Description

Updated car icons from design: https://www.figma.com/file/rrvkvQEoVTOHa7MzyKSaoz/OTP-Global-Design-System?node-id=1364%3A40607

image

image

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Documentation change

How do I test this

  • Add steps to test
  • in bullet point format
  • preferably you can add link to the storybook build in the PR

Checklist

Did you remember to take care of the following?

  • npm i – for new NPM dependencies.
  • npm run lint - to check for linting issues
  • npm run test - to run unit tests
  • npm run test:screenshots - to run snapshot tests

New Feature / Bug Fix

  • Run unit tests to ensure all existing tests are still passing.
  • Add new passing unit tests to cover the code introduced by your pr.

Thanks for contributing!

@hcafaq hcafaq self-assigned this Aug 20, 2022
@github-actions
Copy link

Compiled a new version demo.

@coveralls
Copy link

coveralls commented Aug 20, 2022

Pull Request Test Coverage Report for Build 2932938779

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 95.71%

Totals Coverage Status
Change from base Build 2932866255: 0.0%
Covered Lines: 223
Relevant Lines: 223

💛 - Coveralls

@github-actions
Copy link

github-actions bot commented Aug 20, 2022

Coverage report

❌ An unexpected error occurred. For more details, check console

Error: The process '/usr/bin/git' failed with exit code 1
St.
Category Percentage Covered / Total
🟢 Statements 99.24% 262/264
🟢 Branches 83.75% 67/80
🟢 Functions 100% 33/33
🟢 Lines 100% 223/223

Test suite run success

122 tests passing in 20 suites.

Report generated by 🧪jest coverage report action from 83af439

@hcafaq
Copy link
Contributor Author

hcafaq commented Aug 22, 2022

@derek-heycar I can't see updated icons right now but Icons start showing up if view-box is updated in svg viewBox="0 0 84 50"

@hcafaq hcafaq added the work in progress This is a work in progress label Aug 22, 2022
@hcafaq hcafaq removed the work in progress This is a work in progress label Aug 23, 2022
@hcafaq hcafaq marked this pull request as ready for review August 23, 2022 15:20
@github-actions
Copy link

Compiled a new version demo.

Copy link
Contributor

@derek-heycar derek-heycar left a comment

Choose a reason for hiding this comment

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

Do you check the console log when you developing?

react-dom.development.js:67 Warning: Invalid DOM property `clip-rule`. Did you mean `clipRule`?
react-dom.development.js:67 Warning: Invalid DOM property `fill-rule`. Did you mean `fillRule`?

@hcafaq
Copy link
Contributor Author

hcafaq commented Aug 26, 2022

Do you check the console log when you developing?

react-dom.development.js:67 Warning: Invalid DOM property `clip-rule`. Did you mean `clipRule`?
react-dom.development.js:67 Warning: Invalid DOM property `fill-rule`. Did you mean `fillRule`?

resolved

@github-actions
Copy link

Compiled a new version demo.

@github-actions
Copy link

Compiled a new version demo.

@derek-heycar derek-heycar merged commit 59d3693 into main Aug 26, 2022
@derek-heycar derek-heycar deleted the feature/HEYUI-141-update-svg-icons-for-cars-in-icons-component branch August 26, 2022 10:11
derek-heycar pushed a commit that referenced this pull request Aug 26, 2022
# [1.13.0](v1.12.0...v1.13.0) (2022-08-26)

### Features

* **icons:** added extra icons ([#52](#52)) ([59d3693](59d3693))
@derek-heycar
Copy link
Contributor

🎉 This PR is included in version 1.13.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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants