-
Notifications
You must be signed in to change notification settings - Fork 11
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
Merge I'm Definitely Sober features back into Sobriety #41
Merge I'm Definitely Sober features back into Sobriety #41
Conversation
- If the addiction has never been relapsed, it's not shown at all - The value is correctly and consistently displayed across app restarts
also make the rest of the strings translatable bc i forgor
…r from Main Activity
…s.StartActivityForResult
seriously, that's bad practice
Merging this, thank you |
Done. But... I only told you to use my version, along with yours, and provide feedback. I never expected you to attempt to merge it... |
I know I know but I feel like merging is a good idea |
Thank you so much for helping with Sobriety, I kinda panicked reading that README and I'm so sorry |
...Please don't get mad at me again, I legit panicked |
I'm not mad |
I wasn't mad, I'm sorry I came off that way |
/g |
@unbiaseduser if it's okay, could you send me an email so I can properly apologise? I feel this isn't the place to write a letter. |
I would honestly love for Sobriety to have nightly releases I just want it to have the same name is all, and personally I don't have time to release that often, that's all /genuine,not mad,honest |
Uh, no hard feelings lol. In fact, I am the one who has to apologize for my README writing. (but seriously, i specifically said that)
|
I saw that after I read it and it wasn't as comforting after the fact, but it's okay /g, I just feel really bad for coming off as angry |
@unbiaseduser, whenever you're done with implementing things let me know, so I can merge this.