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

feat: remove references to phone login/number [WPB-10030] #17730

Merged
merged 1 commit into from
Jul 10, 2024

Conversation

tlebon
Copy link
Contributor

@tlebon tlebon commented Jul 9, 2024

TaskWPB-10030 [Web] Remove phone number in user profile

Description

  • removes all phone-number-related login and api usage.
  • removes phone number from account page

Checklist

  • PR has been self reviewed by the author;
  • Hard-to-understand areas of the code have been commented;
  • If it is a core feature, unit tests have been added;

Copy link

sonarcloud bot commented Jul 9, 2024

@codecov-commenter
Copy link

Codecov Report

Attention: Patch coverage is 0% with 3 lines in your changes missing coverage. Please review.

Project coverage is 46.05%. Comparing base (4c2d535) to head (2ebbe13).

Additional details and impacted files
@@            Coverage Diff             @@
##              dev   #17730      +/-   ##
==========================================
- Coverage   46.13%   46.05%   -0.09%     
==========================================
  Files         759      755       -4     
  Lines       24986    24822     -164     
  Branches     5721     5689      -32     
==========================================
- Hits        11528    11431      -97     
+ Misses      12013    11960      -53     
+ Partials     1445     1431      -14     

@tlebon tlebon merged commit 9786a30 into dev Jul 10, 2024
11 checks passed
@tlebon tlebon deleted the feat/remove-phone-login branch July 10, 2024 08:40
PatrykBuniX added a commit that referenced this pull request Jul 11, 2024
* chore: Update translations (#17723)

* chore(deps): bump wire-web-config-default-staging in /app-config (#17725)

Bumps [wire-web-config-default-staging](https://github.com/wireapp/wire-web-config-default) from v0.31.26 to v0.31.27.
- [Commits](wireapp/wire-web-config-default@v0.31.26...c48e253)

---
updated-dependencies:
- dependency-name: wire-web-config-default-staging
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* chore(deps): bump wire-web-config-default-master in /app-config (#17726)

Bumps [wire-web-config-default-master](https://github.com/wireapp/wire-web-config-wire) from v0.31.27-0 to v0.31.28-0.
- [Commits](wireapp/wire-web-config-wire@v0.31.27-0...1d7da3a)

---
updated-dependencies:
- dependency-name: wire-web-config-default-master
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* fix: Hide delivery message for group conversation (#17724)

* fix: Hide delivery message for group conversation

* cr fixes

* feat: Add disclaimer for non production envs (#17727)

* feat: Add disclaimer for non production envs

* add translation for disclaimer

* chore: Update translations (#17728)

* chore: update pipelines [WPB-6227] (#17598)

* chore(wip): update pipelines

* chore: remove staging and prod

* chore: Update translations (#17729)

* feat: remove references to phone login/number (#17730)

* chore: Update translations (#17732)

* fix: replace carriage return with a new line char (#17739)

* fix: replace carraige return with a new line char

* chore: add return type

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: Otto the Bot <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Przemysław Jóźwik <[email protected]>
Co-authored-by: Timothy LeBon <[email protected]>
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.

5 participants