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

Remove notification content extension if not used #19535

Open
guarani opened this issue Oct 28, 2022 · 3 comments
Open

Remove notification content extension if not used #19535

guarani opened this issue Oct 28, 2022 · 3 comments

Comments

@guarani
Copy link
Contributor

guarani commented Oct 28, 2022

There's a notification content extension in the project that doesn't appear to be used. My first thought was that it was used for long-press previews of rich text (images, gifs, etc) in push notifications. @hassaanelgarem pointed out in #19526 (comment) that it's actually probably not used.

We should remove it if it's not used to simplify the Xcode project.

@peril-wordpress-mobile
Copy link

Fails
🚫 Please add a feature label to this issue. e.g. 'Stats'

Generated by 🚫 dangerJS

2 similar comments
@peril-wordpress-mobile
Copy link

Fails
🚫 Please add a feature label to this issue. e.g. 'Stats'

Generated by 🚫 dangerJS

@peril-wordpress-mobile
Copy link

Fails
🚫 Please add a feature label to this issue. e.g. 'Stats'

Generated by 🚫 dangerJS

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant