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

Email user if their photo was overexposed #37

Closed
jtubert opened this issue Nov 3, 2013 · 12 comments
Closed

Email user if their photo was overexposed #37

jtubert opened this issue Nov 3, 2013 · 12 comments
Labels
Milestone

Comments

@jtubert
Copy link
Owner

jtubert commented Nov 3, 2013

When someone overexposed your photo, you will get an email with the over exposed photo.

This is done, let me know if it makes sense how this works now.

@ghost ghost assigned amintorres and jtubert Nov 3, 2013
@amintorres
Copy link
Contributor

Email is fine but then will have to give them a way to turn them off somehow.
I was thinking just an ui alert and we can just show a toggle to turn on and off within the app somewhere.
Thoughts?

@jtubert
Copy link
Owner Author

jtubert commented Nov 3, 2013

Maybe on the profile page we can have an option to turn off email notifications

@amintorres
Copy link
Contributor

Yes!.
I still think it would be nice to do ui alerts.. I thought this was easy with parse no?
If so, yes to both, we should have a way to allow them to turn them both on or off but we should default them both to on to inform the user of such feature.
Best practices will suggest that we include a footnote on each notification email reminding the user that they can turn this off and where they can turn this off within the app.

Lastly, since we are including the pic in the email, we should have either a link to the web pdp for that photo
or, a link for that photo that will launch the app with that photo.
Or both.

If it is not too much work, lets do all of this, if not, lets keep some of this in the back burner.
Will design the settings stuff in the profile next.

@ghost ghost assigned amintorres Nov 4, 2013
@jtubert
Copy link
Owner Author

jtubert commented Nov 4, 2013

I am not sure what you mean by UI alert, do you mean a push notification? Or do you mean like FB that shows you when there is some activity? We can add the link to the app or the web PDP. Whatever you prefer.

@jtubert
Copy link
Owner Author

jtubert commented Nov 4, 2013

The issue with linking to the app is that we don't have a PDP screen in the app. So how do we deep link to it?

@amintorres
Copy link
Contributor

Dammit Amin! get it together man... ;)
Yes, I meant a push notification. I am just trying to get people to come back to the app.
That or we can do the FB like thingie as well to let the user know that some activity has happened that needs their attention.

I hear you on deep linking, lets worry about that later then.
This is what we net it out then.

  • No deep linking.
  • Email stays with the option to turn it off.
  • Push notification when someone exposes over their pic.
  • FB like on top of app icon number counter thingie (1) or (3) etc... depending on how many push notification oen has. They will clear out when the user launches the app?

Does this make sense? Let me know if any of this is too hard to code.

@amintorres
Copy link
Contributor

userProfileSettings.psd for ref.

@jtubert
Copy link
Owner Author

jtubert commented Nov 4, 2013

I will break this up into a few issues since some of them are for me and some for Joe.

@jtubert
Copy link
Owner Author

jtubert commented Nov 4, 2013

Push notification when someone exposes over their pic v- #43 (John)
FB like on top of app icon number counter thingie - #44 (John)
Email stays with the option to turn it off - #45 (Joe/John)

@ghost ghost assigned jtubert Nov 17, 2013
@amintorres
Copy link
Contributor

John, can you confirm if this is completed please?

@jtubert
Copy link
Owner Author

jtubert commented Nov 18, 2013

Are you getting the emails? If so please close

Sent from my iPhone

On Nov 16, 2013, at 11:40 PM, "Amin Torres" notifications@github.com wrote:

John, can you confirm if this is completed please?


Reply to this email directly or view it on GitHub.

@amintorres
Copy link
Contributor

yep!

@jtubert jtubert removed their assignment Oct 31, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants