-
Notifications
You must be signed in to change notification settings - Fork 15
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
feat: inject self profile views monitor - WPB-15038 #2331
Merged
El-Fitz
merged 2 commits into
release/cycle-3.115
from
feat/inject-self-profile-views-monitor
Jan 6, 2025
Merged
feat: inject self profile views monitor - WPB-15038 #2331
El-Fitz
merged 2 commits into
release/cycle-3.115
from
feat/inject-self-profile-views-monitor
Jan 6, 2025
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
echoes-hq
bot
added
echoes: unplanned
Any work item that isn’t part of the product or technical roadmap.
echoes: technical-roadmap/throughput
More specific category, to highlight task aiming at improving the development velocity and effici...
echoes: unplanned/support
Work items completed to meet a customer or user support request.
echoes: technical-roadmap
Work contributing to the Technical Roadmap, to improve our velocity or reduce the technical debt.
echoes: product-roadmap/bug
Work contributing to resolve a bug not critical enough to have raised an incident.
echoes/initiative: personal-users-to-team-users...
labels
Dec 24, 2024
Test Results 1 files 2 suites 1m 56s ⏱️ Results for commit 3dbda2d. ♻️ This comment has been updated with latest results. |
Datadog ReportBranch report: ✅ 0 Failed, 1685 Passed, 26 Skipped, 1m 55.66s Total Time |
caldrian
approved these changes
Jan 3, 2025
wire-ios/Wire-iOS/Sources/UserInterface/SelfProfile/SelfProfileViewsMonitor.swift
Outdated
Show resolved
Hide resolved
wire-ios/Wire-iOS/Sources/UserInterface/SelfProfile/SelfProfileViewsMonitor.swift
Outdated
Show resolved
Hide resolved
samwyndham
approved these changes
Jan 6, 2025
El-Fitz
force-pushed
the
feat/inject-self-profile-views-monitor
branch
from
January 6, 2025 11:50
a8a3b97
to
3dbda2d
Compare
zenkins
pushed a commit
that referenced
this pull request
Jan 6, 2025
3 tasks
El-Fitz
added a commit
that referenced
this pull request
Jan 7, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
echoes/initiative: personal-users-to-team-users...
echoes: product-roadmap/bug
Work contributing to resolve a bug not critical enough to have raised an incident.
echoes: technical-roadmap/throughput
More specific category, to highlight task aiming at improving the development velocity and effici...
echoes: technical-roadmap
Work contributing to the Technical Roadmap, to improve our velocity or reduce the technical debt.
echoes: unplanned/support
Work items completed to meet a customer or user support request.
echoes: unplanned
Any work item that isn’t part of the product or technical roadmap.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Issue
The SelfProfileViewsMonitor is now injected, and a mock has been provided for testing purposes.
Checklist
[WPB-XXX]
.