Skip to content

Refresh the node-app example - #330

Merged
drzraf merged 1 commit into
flowjs:v3from
drzraf:node-app-update
Jan 13, 2021
Merged

drzraf merged 1 commit into
flowjs:v3from
drzraf:node-app-update

Conversation

@drzraf

@drzraf drzraf commented Jan 13, 2021

Copy link
Copy Markdown
Collaborator
  • Update the app' to take into account that Flow.js itsef does not provide anymore a way to know whether a given browser is supported.
  • Compiled code is now into dist/
  • Update to latest dependencies

…ovide anymore

  a way to know whether a given browser is supported.
- Compiled code is now into dist/
- Update to latest dependencies
@drzraf

drzraf commented Jan 13, 2021

Copy link
Copy Markdown
Collaborator Author

I know this change is "hiding" the compatibility-support question, but until it's possible to consistently test the (transpiled) code, it's hard to say (or guess)
As said in the initial v3 commit, it's best to avoid the Flow.js library to depends to much on "window" particular properties and clutter the code with window.navigator property guess work. It's better left to the toolchain.

@drzraf
drzraf merged commit bebdd50 into flowjs:v3 Jan 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant