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

Remove AppTP Beta label in settings #3667

Merged
merged 5 commits into from
Dec 4, 2023

Conversation

CrisBarreiro
Copy link
Contributor

@CrisBarreiro CrisBarreiro commented Oct 13, 2023

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

Description

Steps to test this PR

Feature 1

  • Open settings
  • Check beta label isn't shown anymore

UI changes

combinedsettings

@CrisBarreiro CrisBarreiro force-pushed the feature/cbarreiro/remove-apptp-beta-label branch 2 times, most recently from 72236bd to 161075a Compare November 29, 2023 16:08
CrisBarreiro and others added 5 commits December 4, 2023 17:09
<!--
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/1205217801169320/f
Smartling job:
https://dashboard.smartling.com/app/projects/b01b53ede/account-jobs/b01b53ede:xb9jhwltidzn?filter=ALL&projectIds[]=b01b53ede

See title

_Feature 1_
- [ ] Open AppTP settings
- [ ] Go to my apps
- [ ] Enable one of the apps that are disabled by default
- [ ] Check the banner doesn't contain the word "Beta"

![copymyapps](https://github.com/duckduckgo/Android/assets/6297834/d67b4f7e-e8c4-4f42-8b55-2eefd9422d1e)

---------

Co-authored-by: Marcos Holgado <[email protected]>
<!--
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/1205217801169319/f

### Description
See title

https://dashboard.smartling.com/app/projects/b01b53ede/account-jobs/b01b53ede:82gtdbvplyqu?filter=ALL

### Steps to test this PR

_Feature 1_
- [x] Enable AppTP
- [x] Go to AppTP main view
- [x] Check the banner doesn't start with "During beta"

### UI changes
| Before  | After |
| ------ | ----- |
!(Upload before screenshot)|(Upload after screenshot)|

---------

Co-authored-by: root <[email protected]>
@CrisBarreiro CrisBarreiro force-pushed the feature/cbarreiro/remove-apptp-beta-label branch from e9838fe to ade8c58 Compare December 4, 2023 16:16
@CrisBarreiro CrisBarreiro enabled auto-merge (squash) December 4, 2023 16:17
@CrisBarreiro CrisBarreiro merged commit 5f1fceb into develop Dec 4, 2023
5 checks passed
@CrisBarreiro CrisBarreiro deleted the feature/cbarreiro/remove-apptp-beta-label branch December 4, 2023 16:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants