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 code associated with custom notifications #926

Open
jleach opened this issue Aug 10, 2023 · 1 comment
Open

Remove code associated with custom notifications #926

jleach opened this issue Aug 10, 2023 · 1 comment
Labels
task/technical-debt Technical Debt

Comments

@jleach
Copy link
Contributor

jleach commented Aug 10, 2023

Custom notifications was added to allow teams to add notifications as needed. This behaviour was used by the BC Wallet for inserting a call-to-action to get a Person Credential. This functionality is no longer needed as we add this by creating our own HomeHeaderView override.

This change should be brought up with the community and if it is not used by any teams the dead code should be removed.

@cvarjao cvarjao added the task/technical-debt Technical Debt label Jan 16, 2024
@MosCD3
Copy link
Contributor

MosCD3 commented Nov 8, 2024

Can you give a high level view of the alternative?
Scan QR --> Connection screen --> Offer/Request
What are you thinking of

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
task/technical-debt Technical Debt
Projects
None yet
Development

No branches or pull requests

3 participants