Skip to content
This repository has been archived by the owner on Apr 29, 2023. It is now read-only.

Issue 54, basic SSR without treeshaking location.state and location.actions #67

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jcperez-ch
Copy link

@jcperez-ch jcperez-ch commented Apr 16, 2018

Add a simple check for undefined to the window and history objects to avoid errors when using @hyperapp/router on the server side in combination with @hyperapp/render.

Assuming the subscribe method is only used in the client side.

https://github.com/hyperapp/router/issues/54

@codecov
Copy link

codecov bot commented Apr 16, 2018

Codecov Report

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

Impacted file tree graph

@@          Coverage Diff          @@
##           master    #67   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files           6      6           
  Lines          68     69    +1     
  Branches       11     12    +1     
=====================================
+ Hits           68     69    +1
Impacted Files Coverage Δ
src/location.js 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 8db6747...1b67687. Read the comment docs.

@jcperez-ch jcperez-ch changed the title Issue 54, basic SSR Issue 54, basic SSR without treeshaking location.state and location.actions Apr 16, 2018
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.

1 participant