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.

Sign up for GitHub

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

Jump to bottom

#1752: Avoid duplicate injection on pixiebrix.com #1762

Merged
merged 3 commits into from
Oct 28, 2021
Merged

#1752: Avoid duplicate injection on pixiebrix.com #1762

merged 3 commits into from
Oct 28, 2021

Conversation

fregante
Copy link
Contributor

@fregante fregante commented Oct 27, 2021 •
edited
Loading

@fregante fregante added the bug Something isn't working label Oct 27, 2021
@fregante fregante marked this pull request as ready for review October 27, 2021 17:30
@twschiller twschiller added this to the 1.4.5 milestone Oct 27, 2021
@fregante
Copy link
Contributor Author

Tested now

@twschiller
Copy link
Contributor

🐴 Ponyfill? 😄 fregante/webext-dynamic-content-scripts@v8.0.0...v8.0.1#diff-dcdc3e0b3362edb8fec2a51d3fa51f8fb8af8f70247e06d9887fa934834c9122R13

@fregante fregante merged commit 8270b3c into main Oct 28, 2021
@fregante fregante deleted the F/bug/duplicate-injection branch October 28, 2021 08:31
@fregante
Copy link
Contributor Author

Like a polyfill but equine. Heheh https://github.com/sindresorhus/ponyfill

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@twschiller twschiller twschiller approved these changes

Assignees
No one assigned
Labels
bug Something isn't working
Milestone
1.4.5
Development

Successfully merging this pull request may close these issues.

Context Menu / Sidebar errors / not working when an extension has an all sites match pattern
2 participants
@fregante @twschiller