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

chore(deps): bump web from 0.5.1 to 1.0.0 in /packages/share_plus/share_plus #3105

Merged

Conversation

dependabot[bot]
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Jul 22, 2024

Bumps web from 0.5.1 to 1.0.0.

Release notes

Sourced from web's releases.

package:web v1.0.0

  • Added supertypes' fields to dictionary constructors as dictionaries are allowed to contain those fields.
  • Empty dictionary constructors now create an empty object instead of being treated like non-object literal external constructors.
  • Generate MDN API documentation for getters.
  • Update the docs for generated interface types to reference the MDN Web Docs project.
  • Address several broken links in API documentation.
  • Generate APIs based on if they're on track to be a standard and not experimental. This is a looser restriction from the previous requirement that APIs need to be implemented by Chrome, Firefox, and Safari. As part of this, dictionaries and typedefs are only emitted if they're used by a generated API.
  • Added onUnload event stream to ElementEventGetters extension methods.
  • Expose ElementStream as a public class.
  • Require Dart ^3.4.0.
  • APIs that return a double value now return double instead of num. This is to avoid users accidentally downcasting num, which has different semantics depending on whether you compile to JS or Wasm. See issue #57[] for more details.
  • Fix an issue where some union types didn't account for typedef nullability. #57: dart-lang/web#57
Changelog

Sourced from web's changelog.

1.0.0

  • Added supertypes' fields to dictionary constructors as dictionaries are allowed to contain those fields.
  • Empty dictionary constructors now create an empty object instead of being treated like non-object literal external constructors.
  • Generate MDN API documentation for getters.
  • Update the docs for generated interface types to reference the MDN Web Docs project.
  • Address several broken links in API documentation.
  • Generate APIs based on if they're on track to be a standard and not experimental. This is a looser restriction from the previous requirement that APIs need to be implemented by Chrome, Firefox, and Safari. As part of this, dictionaries and typedefs are only emitted if they're used by a generated API.
  • Added onUnload event stream to ElementEventGetters extension methods.
  • Expose ElementStream as a public class.
  • Require Dart ^3.4.0.
  • APIs that return a double value now return double instead of num. This is to avoid users accidentally downcasting num, which has different semantics depending on whether you compile to JS or Wasm. See issue #57[] for more details.
  • Fix an issue where some union types didn't account for typedef nullability.

#57: dart-lang/web#57

Commits

Dependabot compatibility score

You can trigger a rebase of this PR by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot show <dependency name> ignore conditions will show all of the ignore conditions of the specified dependency
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

@dependabot dependabot bot added dart Pull requests that update Dart code dependencies Pull requests that update a dependency file labels Jul 22, 2024
@miquelbeltran
Copy link
Member

Closes #3093

dependabot bot and others added 2 commits July 23, 2024 21:30
Bumps [web](https://github.com/dart-lang/web) from 0.5.1 to 1.0.0.
- [Release notes](https://github.com/dart-lang/web/releases)
- [Changelog](https://github.com/dart-lang/web/blob/main/CHANGELOG.md)
- [Commits](dart-lang/web@v0.5.1...v1.0.0)

---
updated-dependencies:
- dependency-name: web
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
@vbuberen vbuberen force-pushed the dependabot/pub/packages/share_plus/share_plus/web-1.0.0 branch from a6a40d3 to 4a86dc6 Compare July 23, 2024 18:33
@vbuberen
Copy link
Collaborator

vbuberen commented Jul 23, 2024

I have just updated the code for web: 1.0.0 and also rebased on latest main to include the new override feature.

However, I am still having issues with flutter build web --wasm command:
Screenshot 2024-07-23 at 20 32 24

@miquelbeltran
Copy link
Member

Looks like image_picker_for_web 3.0.3 should support web instead of dart:html, maybe the example needs to use dep overrides: https://pub.dev/packages/image_picker_for_web/changelog#303

@vbuberen
Copy link
Collaborator

Looks like image_picker_for_web 3.0.3 should support web instead of dart:html, maybe the example needs to use dep overrides: https://pub.dev/packages/image_picker_for_web/changelog#303

Yeah, I get it, but how the compilation worked previously when the plugin was migrated to web in #2709? I guess, some changes were introduced when WASM got stable in Flutter 3.22. Because it is not only image_picker, but also url_launcher and some more dependencies. There is a need to upgrade a few more dependencies in the plugin to resolve latest versions that we need for image_picker, etc.

@vbuberen
Copy link
Collaborator

vbuberen commented Jul 24, 2024

Ah, yeah, url_launcher in the plugin itself depends on web: ^0.5.0, so there is an issue with resolving web: 1.0.0.
Will need to wait till dependencies are updated.

Let's wait for this one flutter/packages#7202

dependabot bot and others added 3 commits July 29, 2024 13:49
Bumps [web](https://github.com/dart-lang/web) from 0.5.1 to 1.0.0.
- [Release notes](https://github.com/dart-lang/web/releases)
- [Changelog](https://github.com/dart-lang/web/blob/main/CHANGELOG.md)
- [Commits](dart-lang/web@v0.5.1...v1.0.0)

---
updated-dependencies:
- dependency-name: web
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
@vbuberen vbuberen force-pushed the dependabot/pub/packages/share_plus/share_plus/web-1.0.0 branch from dedde02 to fb3e2bc Compare July 29, 2024 10:49
…us/share_plus/web-1.0.0' into dependabot/pub/packages/share_plus/share_plus/web-1.0.0

# Conflicts:
#	packages/share_plus/share_plus/pubspec.yaml
- Flutter >=3.3.0
- Dart >=2.18.0 <4.0.0
- Flutter >=3.19.0
- Dart >=3.3.0 <4.0.0
Copy link
Collaborator

Choose a reason for hiding this comment

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

Looks like we had outdated info here

@@ -32,7 +32,6 @@
/build/

# Web related
lib/generated_plugin_registrant.dart
Copy link
Collaborator

Choose a reason for hiding this comment

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

This is something edited automatically during the build

});
}
</script>
<script src="main.dart.js" type="application/javascript"></script>
Copy link
Collaborator

Choose a reason for hiding this comment

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

Removed to not see the following message in logs:

Warning: In index.html:39: Manual service worker registration deprecated. Use flutter.js service worker bootstrapping instead.

The change is the same as in flutter/packages#7114

url_launcher_platform_interface: ^2.1.2
ffi: ^2.0.1
web: ^0.5.0
url_launcher_web: ^2.3.2
Copy link
Collaborator

Choose a reason for hiding this comment

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

We only need to update this part of url_launcher to move to web: 1.0.0, but updated the rest as well.

@hnvn
Copy link
Member

hnvn commented Aug 6, 2024

Looking forward this PR

@miquelbeltran
Copy link
Member

Tested by compiling the example with flutter build web, opening a web server from the share_plus/example/build/web folder, and navigating to it from my Android phone using Chrome. Sharing text opened the share sheet as expected.

@vbuberen vbuberen merged commit 1f23910 into main Aug 9, 2024
19 of 22 checks passed
@vbuberen vbuberen deleted the dependabot/pub/packages/share_plus/share_plus/web-1.0.0 branch August 9, 2024 08:10
@jonmountjoy
Copy link

@miquelbeltran Does the dependency have to be so strict? For example on package_info_plus the dependency is web: ">=0.5.0 <2.0.0 which makes updating much easier.

@miquelbeltran
Copy link
Member

In this case, the package uses new APIs available on web so 1.0.0 is a requirement.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dart Pull requests that update Dart code dependencies Pull requests that update a dependency file PR: needs review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants