Skip to content
This repository has been archived by the owner on Feb 20, 2023. It is now read-only.

Remove wrapper for bookmarks management #24214

Closed
mcarare opened this issue Mar 11, 2022 · 1 comment
Closed

Remove wrapper for bookmarks management #24214

mcarare opened this issue Mar 11, 2022 · 1 comment
Assignees
@Alexandru2909
Labels
eng:qa:verified QA Verified Feature:Bookmarks Feature:Telemetry

Comments

@mcarare
Copy link
Contributor

mcarare commented Mar 11, 2022 •
edited by data-sync-user
Loading

Breakout from #21292.

┆Issue is synchronized with this Jira Task
@mcarare mcarare mentioned this issue Mar 11, 2022
Closed
@github-actions github-actions bot added the needs:triage Issue needs triage label Mar 11, 2022
@mcarare mcarare removed the needs:triage Issue needs triage label Mar 11, 2022
@Alexandru2909 Alexandru2909 self-assigned this Mar 23, 2022
Alexandru2909 pushed a commit to Alexandru2909/fenix that referenced this issue Mar 23, 2022
For mozilla-mobile#24214 - Remove Event.wrapper for BookmarksManageme…
a4cdca6 
…nt metrics
Alexandru2909 pushed a commit to Alexandru2909/fenix that referenced this issue Mar 24, 2022
For mozilla-mobile#24214 - Remove Event.wrapper for BookmarksManageme…
dec4f6c 
…nt metrics
Alexandru2909 pushed a commit to Alexandru2909/fenix that referenced this issue Mar 25, 2022
For mozilla-mobile#24214 - Remove Event.wrapper for BookmarksManageme…
764e279 
…nt metrics
Alexandru2909 pushed a commit to Alexandru2909/fenix that referenced this issue Mar 29, 2022
For mozilla-mobile#24214 - Remove Event.wrapper for BookmarksManageme…
ada6a74 
…nt metrics
Alexandru2909 pushed a commit to Alexandru2909/fenix that referenced this issue Mar 31, 2022
For mozilla-mobile#24214 - Remove Event.wrapper for BookmarksManageme…
cafac57 
…nt metrics
Alexandru2909 pushed a commit to Alexandru2909/fenix that referenced this issue Mar 31, 2022
For mozilla-mobile#24214 - Move bookmark removal events to BookmarkCo…
8b02cda 
…ntroller
Alexandru2909 pushed a commit to Alexandru2909/fenix that referenced this issue Mar 31, 2022
For mozilla-mobile#24214 - Move bookmark removal events to BookmarkCo…
2143928 
…ntroller
Alexandru2909 pushed a commit to Alexandru2909/fenix that referenced this issue Mar 31, 2022
For mozilla-mobile#24214 - Move bookmark removal events to BookmarkCo…
1c4064a 
…ntroller
mergify bot pushed a commit that referenced this issue Apr 1, 2022
@mergify
For #24214 - Remove Event.wrapper for BookmarksManagement metrics
0a5b8c4
mergify bot pushed a commit that referenced this issue Apr 1, 2022
@mergify
For #24214 - Move bookmark removal events to BookmarkController
8290129
@Alexandru2909 Alexandru2909 added the eng:qa:needed QA Needed label Apr 1, 2022
@sflorean
Copy link
Contributor

sflorean commented Apr 7, 2022

Tested on latest Nightly with OnePlus 9 Pro (Android 12) and under category "bookmarks_management" the following were displayed:
"edited"
"folder_add"
"copied"
"shared"
"open_in_new_tab"
"open_in_private_tab"
"removed"
"folder_remove"
"moved"
"open"
"bookmarks"
"folder_remove".
Scenarios covered can be found here.

@sflorean sflorean closed this as completed Apr 7, 2022
@sflorean sflorean added eng:qa:verified QA Verified and removed eng:qa:needed QA Needed labels Apr 7, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@Alexandru2909 Alexandru2909

Labels
eng:qa:verified QA Verified Feature:Bookmarks Feature:Telemetry
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@Alexandru2909 @mcarare @sflorean