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

FF to enable filtering notifications by push rules #1412

Merged
merged 3 commits into from
Jul 27, 2023

Conversation

Velin92
Copy link
Member

@Velin92 Velin92 commented Jul 27, 2023

Now that notifications do a small sliding sync when received, they should be able to get the push rules and filter automatically.

Since this is experimental, this needs to be enabled as a FF.

@Velin92 Velin92 requested a review from a team as a code owner July 27, 2023 13:12
@Velin92 Velin92 requested review from pixlwave and removed request for a team July 27, 2023 13:12
@github-actions
Copy link

github-actions bot commented Jul 27, 2023

Warnings
⚠️ Some of the commits are missing ticket numbers. Please consider squashing all commits that don't have a tracking number.
⚠️ You seem to have made changes to views. Please consider adding screenshots.

Generated by 🚫 Danger Swift against b6b9268

@Velin92 Velin92 linked an issue Jul 27, 2023 that may be closed by this pull request
Copy link
Member

@pixlwave pixlwave left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

ElementX/Sources/Application/AppSettings.swift Outdated Show resolved Hide resolved
NSE/Sources/Other/NSESettings.swift Outdated Show resolved Hide resolved
@codecov
Copy link

codecov bot commented Jul 27, 2023

Codecov Report

Patch coverage has no change and project coverage change: +0.42% 🎉

Comparison is base (0e7a48d) 45.49% compared to head (5de6910) 45.91%.

❗ Current head 5de6910 differs from pull request most recent head b6b9268. Consider uploading reports for the commit b6b9268 to get more accurate results

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #1412      +/-   ##
===========================================
+ Coverage    45.49%   45.91%   +0.42%     
===========================================
  Files          396      394       -2     
  Lines        25667    25514     -153     
  Branches     13027    12976      -51     
===========================================
+ Hits         11678    11716      +38     
+ Misses       13695    13500     -195     
- Partials       294      298       +4     
Flag Coverage Δ
unittests 23.01% <0.00%> (+0.24%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Changed Coverage Δ
ElementX/Sources/Application/AppSettings.swift 96.15% <ø> (ø)
...erOptionsScreen/DeveloperOptionsScreenModels.swift 0.00% <ø> (ø)
...perOptionsScreen/View/DeveloperOptionsScreen.swift 0.00% <0.00%> (ø)

... and 10 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Velin92 and others added 3 commits July 27, 2023 15:49
Co-authored-by: Doug <6060466+pixlwave@users.noreply.github.com>
@Velin92 Velin92 force-pushed the mauroromito/FF_push_context_filtering branch from e3a8e45 to b6b9268 Compare July 27, 2023 13:49
@Velin92 Velin92 enabled auto-merge (squash) July 27, 2023 13:49
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@Velin92 Velin92 merged commit 4b14617 into develop Jul 27, 2023
@Velin92 Velin92 deleted the mauroromito/FF_push_context_filtering branch July 27, 2023 14:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Push notifications are unreliable
2 participants