[PR #2955] [MERGED] [PM-5856] No pending requests is visible #23288

Closed
opened 2026-04-16 23:24:05 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/bitwarden/android/pull/2955
Author: @andrebispo5
Created: 1/25/2024
Status: Merged
Merged: 1/30/2024
Merged by: @andrebispo5

Base: mainHead: auth/pm-5856/no-pending


📝 Commits (1)

  • cb4be51 [PM-5856] Add visibility to label.

📊 Changes

1 file changed (+1 additions, -0 deletions)

View changed files

📝 src/App/Pages/Settings/LoginPasswordlessRequestsListPage.xaml (+1 -0)

📄 Description

Type of change

  • Bug fix
  • New feature development
  • Tech debt (refactoring, code cleanup, dependency upgrades, etc)
  • Build/deploy pipeline (DevOps)
  • Other

Objective

In pending login requests the "No pending requests" is visible even when there are requests available.
image

Code changes

Bind label visibility to vm property

Screenshots

image
image

Before you submit

  • Please check for formatting errors (dotnet format --verify-no-changes) (required)
  • Please add unit tests where it makes sense to do so (encouraged but not required)
  • If this change requires a documentation update - notify the documentation team
  • If this change has particular deployment requirements - notify the DevOps team

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/bitwarden/android/pull/2955 **Author:** [@andrebispo5](https://github.com/andrebispo5) **Created:** 1/25/2024 **Status:** ✅ Merged **Merged:** 1/30/2024 **Merged by:** [@andrebispo5](https://github.com/andrebispo5) **Base:** `main` ← **Head:** `auth/pm-5856/no-pending` --- ### 📝 Commits (1) - [`cb4be51`](https://github.com/bitwarden/android/commit/cb4be51d2cefdaba9a4aee109d2b790994570090) [PM-5856] Add visibility to label. ### 📊 Changes **1 file changed** (+1 additions, -0 deletions) <details> <summary>View changed files</summary> 📝 `src/App/Pages/Settings/LoginPasswordlessRequestsListPage.xaml` (+1 -0) </details> ### 📄 Description ## Type of change - [X] Bug fix - [ ] New feature development - [ ] Tech debt (refactoring, code cleanup, dependency upgrades, etc) - [ ] Build/deploy pipeline (DevOps) - [ ] Other ## Objective <!--Describe what the purpose of this PR is. For example: what bug you're fixing or what new feature you're adding--> In pending login requests the "No pending requests" is visible even when there are requests available. ![image](https://github.com/bitwarden/mobile/assets/4648522/f90b31c4-d95b-411d-9111-da785ff95cc5) ## Code changes <!--Explain the changes you've made to each file or major component. This should help the reviewer understand your changes--> <!--Also refer to any related changes or PRs in other repositories--> Bind label visibility to vm property ## Screenshots <!--Required for any UI changes. Delete if not applicable--> ![image](https://github.com/bitwarden/mobile/assets/4648522/42dc4e2d-d6a5-4f84-b009-03946b89ee08) ![image](https://github.com/bitwarden/mobile/assets/4648522/b9a77ecc-a40f-4304-808d-58599e3cc370) ## Before you submit - Please check for formatting errors (`dotnet format --verify-no-changes`) (required) - Please add **unit tests** where it makes sense to do so (encouraged but not required) - If this change requires a **documentation update** - notify the documentation team - If this change has particular **deployment requirements** - notify the DevOps team --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
GiteaMirror added the pull-request label 2026-04-16 23:24:05 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/android#23288