Cosecha / redadalertas

A web app for crowdsourcing immigration raid data.
http://redadalertas.com/
GNU General Public License v3.0
97 stars 24 forks source link

Add Notifications for New Events #65

Closed alexgraffeocohen closed 5 years ago

alexgraffeocohen commented 5 years ago

This PR adds the ability for the Events Map to display notifications when new events are fetched that the Events Map doesn't already have. If there is only one new event, it will tailor the notification to the details of that event.

Also fixes a bug: we accidentally broke the "zoom into the event you just created" feature.

Some limitations of this feature:

ToDo

arghgr commented 5 years ago

Ok, let me know once you're done pulling the Firebase creds out and we should be good to merge! Also, another thing we might want to keep thinking about is, since you've added redux-persist (thanks for figuring that out!), is there any data in the Redux store that we don't want persisted on a user's device? I'll bring this up in the Slack.

alexgraffeocohen commented 5 years ago

@arghgr OK we should be all set now! I removed the plist file from the repository, cleaned up commit history so it's as if I never added it, and added the app center build script to create the plist file during the build process.

Like I said in Slack, all the info in that file is public, so I'm not going to try to reset the credentials, particularly since it's a whole thing (I'd have to delete the Firebase project and re-create it).

I also removed detox, the end to end testing framework I tried, from the repo. It started messing up our App Center builds for some reason, and I want to replace it anyway.

The interesting commits to double check would be c13a0af and 7336638.

arghgr commented 5 years ago

Ok, I've tested this PR and local notifications while the app is open are working, so I'm going to merge. Also added a reminder to add the gitignored file to the dev docs.