search

moxystudio / next-with-moxy

MOXY's boilerplate to accelerate the setup of new Next.js based web applications
https://next-with.moxy.tech
MIT License
101 stars 11 forks source link

chore: move components to react folder #85

Closed ruipneves closed 4 years ago

ruipneves commented 4 years ago

Due to some related imports being performed from different locations (context and corresponding hooks, .e.g) a new folder is created to aggregate all that is react.

The discussion can be found in the following link:

https://moxystudio.slack.com/archives/CGS7BHZM0/p1585155018032800

github-actions[bot] commented 4 years ago

This pull-request can be previewed at https://next-with-moxy-pr-85.now.sh. Please check the Zeit Now status below to see when it's ready.

codecov[bot] commented 4 years ago

Codecov Report

Merging #85 into master will not change coverage. The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff          @@
##           master    #85   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files          14     14           
  Lines          75     75           
  Branches        9      9           
=====================================
  Hits           75     75
Impacted Files Coverage Δ
www/shared/react/footer/Footer.js 100% <ø> (ø)
www/shared/react/cookie-banner/CookieBanner.js 100% <ø> (ø)
www/shared/react/header/Header.js 100% <ø> (ø)
www/app/App.js 100% <ø> (ø) :arrow_up:
www/shared/test-utils/react/app-tree/AppTree.js 100% <ø> (ø)
www/shared/react/main-layout/MainLayout.js 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 4309e0b...f299e19. Read the comment docs.

PedroMiguelSS commented 4 years ago

This one will be closed in favor of #88 .