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

Track when a permission prompt is shown and what action user chooses #12345

Closed
Mugurell opened this issue Jun 17, 2022 · 0 comments · Fixed by #12624
Closed

Track when a permission prompt is shown and what action user chooses #12345

Mugurell opened this issue Jun 17, 2022 · 0 comments · Fixed by #12624
Assignees
Labels
<sitepermissions> Component: feature-sitepermissions
Milestone

Comments

@Mugurell
Copy link
Contributor

Mugurell commented Jun 17, 2022

For https://mozilla-hub.atlassian.net/browse/FNXV2-20626 we also need to track when a permission prompt is shown and what action user chooses.

┆Issue is synchronized with this Jira Task

@Mugurell Mugurell added the <sitepermissions> Component: feature-sitepermissions label Jun 17, 2022
@Mugurell Mugurell self-assigned this Jun 17, 2022
Mugurell added a commit to Mugurell/android-components that referenced this issue Jun 17, 2022
This will allow identifying with what permissions the user interacts.
Mugurell added a commit to Mugurell/android-components that referenced this issue Jun 17, 2022
mergify bot pushed a commit that referenced this issue Jun 23, 2022
This will allow identifying with what permissions the user interacts.
jonalmeida added a commit to jonalmeida/android-components that referenced this issue Aug 9, 2022
…r characters

To avoid matching against issues in other repositories, we can reduce
our matches to not include values if they are preceded by 1 or more
string/number/hypen/underscore values.

Tested using:
 - a JS console:
 ```javascript
 const issueRegExp = new RegExp("(?<![A-Za-z\\-\\_]+)#(\\d+)+", "g");
 const numMatches = Array.from(issueRegExp[Symbol.matchAll]("mozilla-mobile#12345")).length
 const numMatches2 = Array.from(issueRegExp[Symbol.matchAll](" mozilla-mobile#12345 ")).length

 assert(numMatches == 0);
 assert(numMatches2 == 1);
 ```
 - regex tester: https://regexr.com/6rg2v
@mergify mergify bot closed this as completed in #12624 Oct 11, 2022
mergify bot pushed a commit that referenced this issue Oct 11, 2022
To avoid matching against issues in other repositories, we can reduce
our matches to not include values if they are preceded by 1 or more
string/number/hypen/underscore values.

Tested using:
 - a JS console:
 ```javascript
 const issueRegExp = new RegExp("(?<![A-Za-z\\-\\_]+)#(\\d+)+", "g");
 const numMatches = Array.from(issueRegExp[Symbol.matchAll]("#12345")).length
 const numMatches2 = Array.from(issueRegExp[Symbol.matchAll](" #12345 ")).length

 assert(numMatches == 0);
 assert(numMatches2 == 1);
 ```
 - regex tester: https://regexr.com/6rg2v
@github-actions github-actions bot added this to the 107.0.0 🍂 milestone Oct 11, 2022
JohanLorenzo pushed a commit to mozilla-mobile/firefox-android that referenced this issue Oct 13, 2022
…tch issue ID if preceded by other characters

To avoid matching against issues in other repositories, we can reduce
our matches to not include values if they are preceded by 1 or more
string/number/hypen/underscore values.

Tested using:
 - a JS console:
 ```javascript
 const issueRegExp = new RegExp("(?<![A-Za-z\\-\\_]+)#(\\d+)+", "g");
 const numMatches = Array.from(issueRegExp[Symbol.matchAll]("https://github.com/mozilla-mobile/android-components/issues/12345")).length
 const numMatches2 = Array.from(issueRegExp[Symbol.matchAll](" mozilla-mobile/android-components#12345 ")).length

 assert(numMatches == 0);
 assert(numMatches2 == 1);
 ```
 - regex tester: https://regexr.com/6rg2v
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
<sitepermissions> Component: feature-sitepermissions
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant