🐛 Bug Report: Push Notification for iOS badge is string, but should be number #8854
Comments
|
Hello, i would like to help with this |
|
@elprogramadorgt I assigned it to you. Make sure to test it and provide screenshots one finished 😄. |
|
Hello @DH-555 I fixed the issue, but I wasn't able to test it locally due to the connection with redis. Can you help with that? Also, I found some files like swagger2-1.6.x-server.json that have string for the badge, are those automatically generated? I'm asking because if not those need to change as well. |
|
@elprogramadorgt Sure. What problem do you have with the redis connection? I'm not sure about the swagger files. Will check them. |
|
@elprogramadorgt It seems that the swagger files are being generated automatically, so you don't need to change them. |
|
Hello @DH-555, I was trying to run it locally with this command: docker run -it --rm \
--volume /var/run/docker.sock:/var/run/docker.sock \
--volume "$(pwd)"/appwrite:/usr/src/code/appwrite:rw \
--entrypoint="install" \
appwrite/appwrite:1.6.0but I got this error. Do you know how can I test it locally before pushing my PR? |
|
@elprogramadorgt Do you have run appwrite in that machine proveiously? |
|
Can I pick this issue up? |
|
Unassigning due to inactivity. We'll be picking this up. |
|
Hi! Is this issue still open? I'd love to give it a try if no one is working on it. |
|
@stnguyen90 this has been fixed in #9060 no? |
👟 Reproduction steps
This might be related to: #8601
👍 Expected behavior
Looking at the APNS docs, it should be a number instead.
https://developer.apple.com/documentation/usernotifications/generating-a-remote-notification#Payload-key-reference
👎 Actual Behavior
In the Appwrite Docs you can see that the badge requires a string
🎲 Appwrite version
Appwrite Cloud
💻 Operating system
Windows
🧱 Your Environment
1.6.x - Cloud
👀 Have you spent some time to check if this issue has been raised before?
🏢 Have you read the Code of Conduct?
The text was updated successfully, but these errors were encountered: