Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

web-sys master merge #966

Merged
merged 44 commits into from
Feb 26, 2020
Merged

web-sys master merge #966

merged 44 commits into from
Feb 26, 2020

Conversation

daxpedda
Copy link
Contributor

@daxpedda daxpedda commented Feb 25, 2020

So here I thought I was extremely smart by merging commits to not end up having to rebase, turns out you can't really squash merge commits.

If somebody has a suggestion how to clean this up I'm all ears.

Additionally I removed most Options in services, as removing cancel from Task made those absolute. Somebody correct me if this was wrong.

See #961.

jstarry and others added 30 commits January 26, 2020 17:36
This allows the various formats (e.g., CBOR) that can't be converted
in or out of Text to be used in a websocket by specifying you want to
use a binary-only connection.  A text-only connection is provided for
symmetry.
In order to avoid misuse It was decided to remove `cancel` from `Task` trait
and delegate such logic to `Drop`.
* fix native_worker.js 404 error

* fix contribute link

* drops contribute link
* yew-macro: Simplify Properties validation

* Fix most clippy warnings
* wip

* Fix component rendering process
…ront (#927)

* Fix wrong order of classes when reordering or adding classes in the front/middle (#926)

* Fix coding style (#926)

* Use set_attribute(class, className) for classes (#926)
* Clean up README

* Remove bors
* Implement more of FetchOptions

Add referrer, referrer_policy and integrity

* Pass options to Request constructor instead of fetch

* Derive Debug on ReferrerPolicy and Referrer

* WIP: Add tests

* Add tests for referrer and redirect

* Add FetchService `integrity` field test

* Add FetchService `integrity` field fail test and refactor

* Add FetchService `referrer_policy` field tests

Co-authored-by: Justin Starry <justin.m.starry@gmail.com>
Signed-off-by: Matthew Fisher <matt.fisher@microsoft.com>
This add a small example of how to use agents in order to send
messages between components.
@jstarry
Copy link
Member

jstarry commented Feb 26, 2020

So here I thought I was extremely smart by merging commits to not end up having to rebase, turns out you can't really squash merge commits.
If somebody has a suggestion how to clean this up I'm all ears.

This is fine with me, adding more merge commits at this point is not really hurting anything.

Additionally I removed most Options in services, as removing cancel from Task made those absolute. Somebody correct me if this was wrong.

Lgtm

@jstarry jstarry merged commit 4b43e24 into yewstack:web-sys Feb 26, 2020
@daxpedda daxpedda deleted the web-sys-fixes branch February 26, 2020 05:33
@jetli
Copy link
Contributor

jetli commented Feb 26, 2020

This is huge and really cool :)

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.