Skip to content

[PS-1353] Add support for the Neeva app#2045

Merged
djsmith85 merged 3 commits intobitwarden:masterfrom
dan-alcantara-neeva:master
Dec 28, 2022
Merged

[PS-1353] Add support for the Neeva app#2045
djsmith85 merged 3 commits intobitwarden:masterfrom
dan-alcantara-neeva:master

Conversation

@dan-alcantara-neeva
Copy link
Copy Markdown

Type of change

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

Objective

Add support for the Neeva app.
https://community.bitwarden.com/t/adding-support-on-android-for-the-neeva-browser/43757

Code changes

Copied and tweaked https://github.com/bitwarden/mobile/pull/1855/files:

  • AccessibilityHelpers.cs was adjusted to be able to find the View ID of a TextView that contains the full URL of the page.
  • AutofillHelpers.cs and autofillservice.xml were edited to allow working with the Neeva app's package.

Before you submit

  • I have checked for formatting errors (dotnet tool run dotnet-format --check) (required)
  • I have added unit tests where it makes sense to do so (encouraged but not required)
  • This change requires a documentation update (notify the documentation team)
  • This change has particular deployment requirements (notify the DevOps team)

@bitwarden-bot
Copy link
Copy Markdown

Thank you for your contribution! We've added this to our internal Community PR board for review.
ID: PS-1353

@bitwarden-bot bitwarden-bot changed the title Add support for the Neeva app [PS-1353] Add support for the Neeva app Aug 22, 2022
Copy link
Copy Markdown

@ArthurFriedenreich ArthurFriedenreich left a comment

Choose a reason for hiding this comment

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

All good

@ArthurFriedenreich
Copy link
Copy Markdown

It's about time to get Neeva browser registered in Bitwarden.

@djsmith85 djsmith85 self-assigned this Dec 28, 2022
Copy link
Copy Markdown
Contributor

@djsmith85 djsmith85 left a comment

Choose a reason for hiding this comment

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

@dan-alcantara-neeva Thank you for your contribution. The changes are looking good and will be included in the next mobile release.

@djsmith85 djsmith85 merged commit cbccd10 into bitwarden:master Dec 28, 2022
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.

4 participants