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

Use FLAG_IMMUTABLE or FLAG_MUTABLE on PendingIntents to support Android API 31+ #11666

Closed
Mugurell opened this issue Feb 4, 2022 · 3 comments
Closed

Use FLAG_IMMUTABLE or FLAG_MUTABLE on PendingIntents to support Android API 31+ #11666

Mugurell opened this issue Feb 4, 2022 · 3 comments
Assignees
@Mugurell

Comments

@Mugurell
Copy link
Contributor

Mugurell commented Feb 4, 2022 •
edited by data-sync-user
Loading

Official documentation - https://developer.android.com/about/versions/12/behavior-changes-12#pending-intent-mutability

Following #11563 and mozilla-mobile/focus-android#6346 seems like there are still places which needs updates.

┆Issue is synchronized with this Jira Task
@Mugurell Mugurell self-assigned this Feb 4, 2022
@Mugurell Mugurell mentioned this issue Feb 4, 2022
Closed
Mugurell added a commit to Mugurell/android-components that referenced this issue Feb 4, 2022
@Mugurell
For mozilla-mobile#11666 - Add PendingUtils.defaultFlags helper
618bae7
Mugurell added a commit to Mugurell/android-components that referenced this issue Feb 4, 2022
@Mugurell
For mozilla-mobile#11666 - Use PendingUtils.defaultFlags to specify m…
0eff6a2 
…utability
@Mugurell Mugurell mentioned this issue Feb 4, 2022
Closed
Mugurell added a commit to Mugurell/android-components that referenced this issue Feb 7, 2022
@Mugurell
For mozilla-mobile#11666 - Add PendingUtils.defaultFlags helper
342f899
Mugurell added a commit to Mugurell/android-components that referenced this issue Feb 7, 2022
@Mugurell
For mozilla-mobile#11666 - Use PendingUtils.defaultFlags to specify m…
170e4b6 
…utability
mergify bot pushed a commit that referenced this issue Feb 7, 2022
@Mugurell @mergify
For #11666 - Add PendingUtils.defaultFlags helper
39f8dd6
mergify bot pushed a commit that referenced this issue Feb 7, 2022
@Mugurell @mergify
For #11666 - Use PendingUtils.defaultFlags to specify mutability
ec02018
@Amejia481
Copy link
Contributor

Amejia481 commented Mar 1, 2022 •
edited
Loading

👋🏽
@Mugurell as we merged a couple of commit related this ticket, is OK to close it? Or are there pending changes? :)

@Mugurell
Copy link
Contributor Author

Mugurell commented Mar 1, 2022

Last PR should've addressed all remaining cases so I think the work for this is completed.
I'll close the ticket. Thank you!

@Mugurell Mugurell closed this as completed Mar 1, 2022
@Amejia481
Copy link
Contributor

Thanks for the update :)

@KazaHesto KazaHesto mentioned this issue Mar 11, 2022
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@Mugurell Mugurell

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@Amejia481 @Mugurell