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 local firebase plugin #169

Merged

Conversation

jamshale
Copy link
Contributor

@jamshale jamshale commented Aug 6, 2024

This should be all it takes to remove the local plugin from this project in favor of the plugin from the official plugin repo. Used the latest plugin version 0.12.2, which is on python 3.9.

NOTE: I need to test this better but I can see the plugin getting loaded and initialized.

@jamshale jamshale marked this pull request as ready for review August 6, 2024 17:10
@swcurran
Copy link
Contributor

Approved for merging pending your testing. Good stuff!

@jamshale
Copy link
Contributor Author

jamshale commented Aug 12, 2024

I have confirmed the plugin is loaded and trying to send the request to firebase. I didn't setup everything with my own firebase account to actually get to notification but I'm confident the plugin is working as expected.

It should still get tested out from dev and test before being promoted to prod.

@jamshale jamshale merged commit 6033586 into openwallet-foundation:main Aug 12, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants