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.

Sign up for GitHub

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

Jump to bottom

Update known to collect icons #3671

Merged
merged 2 commits into from
Oct 20, 2023
Merged

Update known to collect icons #3671

merged 2 commits into from
Oct 20, 2023

Conversation

CrisBarreiro
Copy link
Contributor

@CrisBarreiro CrisBarreiro commented Oct 13, 2023 •
edited by aitorvs
Loading

Task/Issue URL: https://app.asana.com/0/0/1205176011518410/f

Description

Update icons in "Known to collect"

Steps to test this PR

Feature 1

  • Enable AppTP
  • Open an app that uses trackers
  • Get back to AppTP and check blocked trackers
  • Click on a specific trackers
  • Check we're displaying the new icons

UI changes

knowncollect

This was referenced Oct 13, 2023
Merged
Merged
Closed
Merged
@CrisBarreiro
Copy link
Contributor Author

CrisBarreiro commented Oct 13, 2023 •
edited
Loading

Current dependencies on/for this PR:

This comment was auto-generated by Graphite.

@CrisBarreiro CrisBarreiro force-pushed the feature/cbarreiro/update-notification-icons branch from 12dfa69 to ebb8f55 Compare October 13, 2023 15:49
@CrisBarreiro CrisBarreiro force-pushed the feature/cbarreiro/update-known-to-collect-icons branch from 86576b0 to 673600e Compare October 13, 2023 15:49
@CrisBarreiro CrisBarreiro force-pushed the feature/cbarreiro/update-notification-icons branch from ebb8f55 to c054146 Compare October 16, 2023 07:32
@CrisBarreiro CrisBarreiro force-pushed the feature/cbarreiro/update-known-to-collect-icons branch from 673600e to a774819 Compare October 16, 2023 07:32
This was referenced Oct 16, 2023
Merged
Merged
Merged
@CrisBarreiro CrisBarreiro mentioned this pull request Oct 16, 2023
Merged
3 tasks
aitorvs
aitorvs approved these changes Oct 16, 2023
View reviewed changes
Copy link
Collaborator

@aitorvs aitorvs left a comment

Choose a reason for hiding this comment

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

LGTM

@CrisBarreiro CrisBarreiro force-pushed the feature/cbarreiro/update-notification-icons branch from c054146 to 46ee896 Compare October 18, 2023 07:17
@CrisBarreiro CrisBarreiro force-pushed the feature/cbarreiro/update-known-to-collect-icons branch from a774819 to 2e544cd Compare October 18, 2023 07:17
@CrisBarreiro CrisBarreiro force-pushed the feature/cbarreiro/update-notification-icons branch from 46ee896 to 96d5eeb Compare October 18, 2023 14:31
@CrisBarreiro CrisBarreiro force-pushed the feature/cbarreiro/update-known-to-collect-icons branch from 2e544cd to 2fd11cf Compare October 18, 2023 14:31
@CrisBarreiro CrisBarreiro mentioned this pull request Oct 18, 2023
Merged
14 tasks
@CrisBarreiro CrisBarreiro force-pushed the feature/cbarreiro/update-notification-icons branch from 96d5eeb to 3935c51 Compare October 19, 2023 11:20
@CrisBarreiro CrisBarreiro force-pushed the feature/cbarreiro/update-known-to-collect-icons branch from 2fd11cf to 7e108b8 Compare October 19, 2023 11:20
@CrisBarreiro CrisBarreiro force-pushed the feature/cbarreiro/update-notification-icons branch from 3935c51 to 6fef5cf Compare October 19, 2023 11:34
@CrisBarreiro CrisBarreiro force-pushed the feature/cbarreiro/update-known-to-collect-icons branch from 7e108b8 to 5484e3d Compare October 19, 2023 11:56
@CrisBarreiro CrisBarreiro force-pushed the feature/cbarreiro/update-notification-icons branch from 6fef5cf to a264019 Compare October 19, 2023 11:57
@CrisBarreiro CrisBarreiro force-pushed the feature/cbarreiro/update-known-to-collect-icons branch from 5484e3d to 4883c03 Compare October 19, 2023 11:57
@CrisBarreiro CrisBarreiro force-pushed the feature/cbarreiro/update-notification-icons branch from a264019 to a9dcc47 Compare October 19, 2023 12:04
@CrisBarreiro CrisBarreiro force-pushed the feature/cbarreiro/update-known-to-collect-icons branch from 4883c03 to 4378f46 Compare October 19, 2023 12:05
@CrisBarreiro
Move learn more text to the next line and make it bold (#3683)
ae9ecd2 
<!--
Note: This checklist is a reminder of our shared engineering
expectations.
The items in Bold are required
If your PR involves UI changes:
1. Upload screenshots or screencasts that illustrate the changes before
/ after
2. Add them under the UI changes section (feel free to add more columns
if needed)
If your PR does not involve UI changes, you can remove the **UI
changes** section

At a minimum, make sure your changes are tested in API 23 and one of the
more recent API levels available.
-->

Task/Issue URL: https://app.asana.com/0/0/1205217801169325/f 

### Description
See title

### Steps to test this PR

_Feature 1_
- [x] Clear app data
- [x] Enable AppTP
- [x] Go to step 3 of the onboarding
- [x] Check the "Learn More" text is shown in a separate line and in
bold

### UI changes

![learnmore](https://github.com/duckduckgo/Android/assets/6297834/97fc78fe-96fc-4025-8052-ff92740459ab)
@CrisBarreiro CrisBarreiro merged commit b451821 into feature/cbarreiro/update-notification-icons Oct 20, 2023
@CrisBarreiro CrisBarreiro deleted the feature/cbarreiro/update-known-to-collect-icons branch October 20, 2023 11:23
@CrisBarreiro CrisBarreiro mentioned this pull request Oct 20, 2023
Merged
CrisBarreiro added a commit that referenced this pull request Nov 6, 2023
@CrisBarreiro
AppTP - minor UX improvements (#3668)
4bf1f70 
* **PR #3670** <a
href="https://app.graphite.dev/github/pr/duckduckgo/Android/3670"
target="_blank"><img
src="https://static.graphite.dev/graphite-32x32-black.png"
alt="Graphite" width="10px" height="10px"/></a>
* **PR #3671** <a
href="https://app.graphite.dev/github/pr/duckduckgo/Android/3671"
target="_blank"><img
src="https://static.graphite.dev/graphite-32x32-black.png"
alt="Graphite" width="10px" height="10px"/></a>
* **PR #3683** <a
href="https://app.graphite.dev/github/pr/duckduckgo/Android/3683"
target="_blank"><img
src="https://static.graphite.dev/graphite-32x32-black.png"
alt="Graphite" width="10px" height="10px"/></a>
* **PR #3684** <a
href="https://app.graphite.dev/github/pr/duckduckgo/Android/3684"
target="_blank"><img
src="https://static.graphite.dev/graphite-32x32-black.png"
alt="Graphite" width="10px" height="10px"/></a>
* **PR #3686** <a
href="https://app.graphite.dev/github/pr/duckduckgo/Android/3686"
target="_blank"><img
src="https://static.graphite.dev/graphite-32x32-black.png"
alt="Graphite" width="10px" height="10px"/></a>
* **PR #3693** <a
href="https://app.graphite.dev/github/pr/duckduckgo/Android/3693"
target="_blank"><img
src="https://static.graphite.dev/graphite-32x32-black.png"
alt="Graphite" width="10px" height="10px"/></a>
* **PR #3715** <a
href="https://app.graphite.dev/github/pr/duckduckgo/Android/3715"
target="_blank"><img
src="https://static.graphite.dev/graphite-32x32-black.png"
alt="Graphite" width="10px" height="10px"/></a>
* **PR #3732** <a
href="https://app.graphite.dev/github/pr/duckduckgo/Android/3732"
target="_blank"><img
src="https://static.graphite.dev/graphite-32x32-black.png"
alt="Graphite" width="10px" height="10px"/></a>
joshliebe pushed a commit that referenced this pull request Nov 7, 2023
@CrisBarreiro @joshliebe
AppTP - minor UX improvements (#3668)
915e1e6 
* **PR #3670** <a
href="https://app.graphite.dev/github/pr/duckduckgo/Android/3670"
target="_blank"><img
src="https://static.graphite.dev/graphite-32x32-black.png"
alt="Graphite" width="10px" height="10px"/></a>
* **PR #3671** <a
href="https://app.graphite.dev/github/pr/duckduckgo/Android/3671"
target="_blank"><img
src="https://static.graphite.dev/graphite-32x32-black.png"
alt="Graphite" width="10px" height="10px"/></a>
* **PR #3683** <a
href="https://app.graphite.dev/github/pr/duckduckgo/Android/3683"
target="_blank"><img
src="https://static.graphite.dev/graphite-32x32-black.png"
alt="Graphite" width="10px" height="10px"/></a>
* **PR #3684** <a
href="https://app.graphite.dev/github/pr/duckduckgo/Android/3684"
target="_blank"><img
src="https://static.graphite.dev/graphite-32x32-black.png"
alt="Graphite" width="10px" height="10px"/></a>
* **PR #3686** <a
href="https://app.graphite.dev/github/pr/duckduckgo/Android/3686"
target="_blank"><img
src="https://static.graphite.dev/graphite-32x32-black.png"
alt="Graphite" width="10px" height="10px"/></a>
* **PR #3693** <a
href="https://app.graphite.dev/github/pr/duckduckgo/Android/3693"
target="_blank"><img
src="https://static.graphite.dev/graphite-32x32-black.png"
alt="Graphite" width="10px" height="10px"/></a>
* **PR #3715** <a
href="https://app.graphite.dev/github/pr/duckduckgo/Android/3715"
target="_blank"><img
src="https://static.graphite.dev/graphite-32x32-black.png"
alt="Graphite" width="10px" height="10px"/></a>
* **PR #3732** <a
href="https://app.graphite.dev/github/pr/duckduckgo/Android/3732"
target="_blank"><img
src="https://static.graphite.dev/graphite-32x32-black.png"
alt="Graphite" width="10px" height="10px"/></a>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@aitorvs aitorvs aitorvs approved these changes

Assignees

@aitorvs aitorvs

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@CrisBarreiro @aitorvs