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

Fix #233 #234

Merged
merged 2 commits into from
May 22, 2023
Merged

Fix #233 #234

merged 2 commits into from
May 22, 2023

Conversation

fonsp
Copy link
Contributor

@fonsp fonsp commented May 17, 2023

Tested locally: 🎉

Scherm­afbeelding 2023-05-17 om 12 48 11

@mozfreddyb
Copy link
Collaborator

I'll happily take this patch, once it passes the linter, but I want to note that the library is completely out of sync with the specification and we are likely going to break a lot of things soon. I don't recommend using it anywhere near production :))

@fonsp
Copy link
Contributor Author

fonsp commented May 17, 2023

Alright, thanks for the heads-up!

@fonsp
Copy link
Contributor Author

fonsp commented May 18, 2023

Should be good now! Maybe put that warning in the readme also?

@mozfreddyb
Copy link
Collaborator

Should be good now! Maybe put that warning in the readme also?

My bad. I thought we had that.

@mozfreddyb mozfreddyb merged commit ac9e8fc into mozilla:main May 22, 2023
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.

2 participants