Skip to content
This repository has been archived by the owner on Jan 16, 2022. It is now read-only.

Not Found Component #170

Merged
merged 8 commits into from Oct 12, 2019
Merged

Not Found Component #170

merged 8 commits into from Oct 12, 2019

Conversation

priscilawebdev
Copy link
Contributor

Type: Refactor

The following has been addressed in the PR: There is not issue for this PR

Description: I had to refactor this component to correct the testing of the Version page. I ended up simplifying the component.

@codecov
Copy link

codecov bot commented Oct 9, 2019

Codecov Report

Merging #170 into master will increase coverage by 0.55%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #170      +/-   ##
==========================================
+ Coverage    88.2%   88.75%   +0.55%     
==========================================
  Files         113      113              
  Lines         924      916       -8     
  Branches      147      162      +15     
==========================================
- Hits          815      813       -2     
+ Misses         96       89       -7     
- Partials       13       14       +1
Impacted Files Coverage Δ
src/utils/styles/spacings.ts 100% <ø> (ø)
src/components/NotFound/NotFound.tsx 90% <100%> (+43.84%) ⬆️
src/components/DetailSidebar/DetailSidebar.tsx 85% <0%> (ø) ⬆️

package.json Outdated Show resolved Hide resolved
Copy link
Member

@juanpicado juanpicado left a comment

Choose a reason for hiding this comment

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

nice .... 👏
Screen Shot 2019-10-10 at 9 50 03 PM

@juanpicado juanpicado merged commit fdbdb63 into master Oct 12, 2019
@delete-merged-branch delete-merged-branch bot deleted the refactor/update_not_found_component branch October 12, 2019 11:23
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
2 participants