8000 fix: apns provider key file upload input by stnguyen90 · Pull Request #2092 · appwrite/console · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

fix: apns provider key file upload input #2092

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

Merged
merged 1 commit into from
Jul 8, 2025
Merged

Conversation

stnguyen90
Copy link
Contributor

What does this PR do?

Don't hardcode the upload dropzone to be for service account JSON files only.

Fixes #2045

Test Plan

Tested locally:

FCM:

image

APNS:

image

Related PRs and Issues

Have you read the Contributing Guidelines on issues?

Yes

Don't hardcode the upload dropzone to be for service account JSON files only.
@stnguyen90 stnguyen90 requested a review from ItzNotABug July 7, 2025 23:34
@stnguyen90 stnguyen90 moved this to Code Review in 1.7 Release Jul 7, 2025
@stnguyen90
Copy link
Contributor Author
stnguyen90 commented Jul 7, 2025

One issue I'm noticing is the popover isn't updating correct. For example, after opening the APNS Team ID popover and then switching to FCM, the 1st popover shows the Team ID popover:

image

Any ideas?

@stnguyen90 stnguyen90 self-assigned this Jul 8, 2025
@ItzNotABug
Copy link
Member

One issue I'm noticing is the popover isn't updating correct. For example, after opening the APNS Team ID popover and then switching to FCM, the 1st popover shows the Team ID popover:

image Any ideas?

could be a reactivity issue. maybe try with a #key block with a var that's reactive.

@ItzNotABug ItzNotABug merged commit c8057c8 into main Jul 8, 2025
2 checks passed
@ItzNotABug ItzNotABug deleted the fix-apns-provider-key branch July 8, 2025 13:26
@github-project-automation github-project-automation bot moved this from Code Review to Done in 1.7 Release Jul 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

🐛 Bug Report: Unable to create APNS provider
2 participants
0