Skip to content

Use functions from gloo_utils instead of re-implementing them #2124

Merged
voidpumpkin merged 5 commits intoyewstack:masterfrom
ranile:gloo-utils-fns
Nov 11, 2021
Merged

Use functions from gloo_utils instead of re-implementing them #2124
voidpumpkin merged 5 commits intoyewstack:masterfrom
ranile:gloo-utils-fns

Conversation

@ranile
Copy link
Member

@ranile ranile commented Oct 29, 2021

Description

This PR:

  • Replaces web-sys helper functions in yew::utils with those from gloo_utils
  • Gets rid of anyhow dependency

Fixes #2058
Fixes #925

Checklist

  • I have run cargo make pr-flow
  • I have reviewed my own code
  • I have added tests

@github-actions
Copy link

github-actions bot commented Oct 29, 2021

Visit the preview URL for this PR (updated for commit ff211e3):

https://yew-rs--pr2124-gloo-utils-fns-m9r6wkwq.web.app

(expires Thu, 18 Nov 2021 12:15:16 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

voidpumpkin
voidpumpkin previously approved these changes Nov 11, 2021
Copy link
Member

@voidpumpkin voidpumpkin left a comment

Choose a reason for hiding this comment

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

Minor comment, but LGTM

@voidpumpkin voidpumpkin added A-examples Area: The examples A-yew Area: The main yew crate A-yew-agent Area: The yew-agent crate A-yew-router Area: The yew-router crate labels Nov 11, 2021
# Conflicts:
#	packages/yew-router/src/router.rs
#	packages/yew-router/src/service.rs
#	packages/yew-router/src/utils.rs
#	packages/yew-router/tests/router.rs
#	website/docs/concepts/function-components/pre-defined-hooks.md
@voidpumpkin voidpumpkin merged commit dd2db93 into yewstack:master Nov 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-examples Area: The examples A-yew Area: The main yew crate A-yew-agent Area: The yew-agent crate A-yew-router Area: The yew-router crate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Use functions from gloo_utils instead of re-implementing them Replace uses of anyhow with error types

4 participants