-
-
Notifications
You must be signed in to change notification settings - Fork 412
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
Added Parse Platform Support #212
Conversation
Codecov Report
@@ Coverage Diff @@
## master #212 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 84 85 +1
Lines 10533 10625 +92
Branches 1759 1772 +13
=========================================
+ Hits 10533 10625 +92
Continue to review full report at Codecov.
|
6c664a1
to
1838fec
Compare
Thanks @caronc - maybe @lord-aerion can help, since it was his comment that lead me to parse platform: |
I'm not sure I can be of much help as I'm not a developer. I have a Gihub account for other reasons. Getting into development is a pipe dream, really… My comment re: Parse Server came from some research I'd done when trying to find an open source alternative to Google Firebase, which is often cited as a reason why certain projects (such as Joplin) can't be fully open sourced. This hinders adoption by F-Droid. Testing, however, is something I'd be happy to assist with. |
# Create a virtual environment to work in
# This way you can just destroy it after when it's all over.
# It will create a directory called apprise
python3 -m venv apprise
# Change into our new directory
cd apprise
# Activate our virtual environment
source bin/activate
# Install our branch:
pip install git+https://github.com/caronc/apprise.git@208-parse-platform-support
# Test'r out! :)
# Substitute {app_id} for your actual App ID
# Substitute {master_key} for your actual Master Key
# Substitute {hostname} for your actual Parse Platform Server you have installed
apprise -t "Title" -b "Body" parsep://{app_id}:{master_key}@{hostname}
# That will hopefully pass along notifications.
# Where apprise excels is you might not really want to put all of that
# information on your command line. It's a pain in the butt .. but
# you can also just create a configuration file and put it in there.
# Open up any text file at all (brand new one) and put this in it
# (obviously subbing out all credentials and hostname for your own):
# parsep://{app_id}:{master_key}@{hostname}
# Then test apprise:
apprise -t "Title" -b "Body" -c your_config_file_you_created.txt If you place your config file in special spots, you don't even have to specify the config file. it will just be read in all the time. But for now you don't need to worry about that. If you can help pass along that it works (or doesn't); that would be absolutely awesome. No pressure though of course! 🙂 |
I’m free this weekend to test this PR. |
4926a03
to
93efa2d
Compare
@caronc Sorry for the super late reply. I tried the branch out and it worked. I was able to send push notifications. Thank you for taking time to implement this. I can do a separate PR to improve the payload. (deviceToken, pushType etc). |
1838fec
to
3cb579e
Compare
3cb579e
to
1dc5b57
Compare
@dplewis no problem! Thank you for testing it; content is now merged into the master. |
Description:
Related issue (if applicable): fixes #208
Syntax:
parsep://{app_id}:{master_key}@{hostname}
parseps://{app_id}:{master_key}@{hostname}
Note:
parsep://
is for insecure (http://) where asparseps://
is for secure (https://) connections.By default, it will notify both the
ios
andandroid
configuration you have set up with the Parse Platform Setup. If you wish to override this and only notify one, you can use thedevice
directive:parsep://{app_id}:{master_key}@{hostname}?device=android
parsep://{app_id}:{master_key}@{hostname}?device=ios
There is no need to specify
/parse/push
in the pparse:// URL's as it is implied.New Service Completion Status
keywords
section of thesetup()
declaration%global common_description
Checklist
flake8
)