Skip to content
This repository has been archived by the owner on Nov 4, 2024. It is now read-only.

Update code for js_interop #42

Merged
merged 2 commits into from
Aug 2, 2024
Merged

Conversation

svenopdehipt
Copy link
Contributor

@svenopdehipt svenopdehipt commented Apr 18, 2024

Replaced dart.library.html with dart.library.js_interop to support the new javascript interop (https://dart.dev/interop/js-interop and https://dart.dev/interop/js-interop/package-web#conditional-imports).

Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Hi 👋🏽 Thank you for opening your first PR with simpleclub/url_strategy ❤

You can expect a review from us soon ☺️
In the meantime, please check our contribution guidelines, the PR checklist, and the PR checks.

@svenopdehipt
Copy link
Contributor Author

Do I have to add something to make this mergable?

@Sese-Schneider
Copy link
Member

@svenopdehipt thank you for your contribution!

@Sese-Schneider Sese-Schneider merged commit 305816b into simpleclub:main Aug 2, 2024
5 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants