+- UserSpice (https://userspice.com/forums)
+-- Forum: Support Center (https://userspice.com/forums/forumdisplay.php?fid=23)
+--- Forum: UserSpice 4.3 and Below (https://userspice.com/forums/forumdisplay.php?fid=26)
+--- Thread: Status Update 016 (/showthread.php?tid=753)
Status Update 016 - Brandin - 09-16-2017
@mudmin where should I add the logger? In the after login script in usersc or should I build this into login.php?
Status Update 016 - karsen - 09-16-2017
I have the GitHub Desktop app but it's not intuitive at all. Helpful but not everything it should be. I'll check out SourceTree.
I do want to tweak the notifications a bit before we finish 4.3 (hopefully today).
Status Update 016 - Brandin - 09-16-2017
Sounds good to me
With SourceTree I just started my own branch and pushed that, so Mudmin can merge it
Status Update 016 - mudmin - 09-16-2017
I do it all from the command line. Merging isnt' too hard. Dealing with merge conflicts can be a little tricky. What's your git username, @Karsen?
For what you're doing it's easy enough to do in the command line.
cd into the folder that has the repo in it
git checkout -b karsen
then when you are done making your changes
git add .
git commit -m "your description"
git push origin karsen
Status Update 016 - karsen - 09-16-2017
It's karsen777 (same for BitBucket). I'm currently falling in love with Atom, it autodetects my changes!
Status Update 016 - mudmin - 09-16-2017
I love it.
I also added you as a contributor
Status Update 016 - Brandin - 09-16-2017
My friend got me addicted to it! I was using Notepad++ for a long while but Atom is awesome!
Status Update 016 - Brandin - 09-17-2017
Alright, to get us on our game for this:
-Deploy @karsen from #6122 post in this thread
-Add a setting to enable/disable notifications (I can tackle this if nobody else wants to?)
-I have a bugfix for loggers on admin.php
-Deploy login in the login function (somewhere - not sure where yet)
-Stripe removal??? (as discussed in another thread)
-Admin notification panel (I can tackle this if nobody else wants to?)
I think that should be it and anything else @mudmin wants
Status Update 016 - mudmin - 09-17-2017
Ok. I want to find out if @Karsen started his work in another branch. I don't want to conflict with him.
I'm really good with leaving the stripe fields for upgraders and just not having them in for new users. My new implementations don't use them that way anyway. I have one big audit table for stripe transactions, not a per user customer number.
I'll take a look at the login thing.
Status Update 016 - mudmin - 09-17-2017
I was also helping KingNath on his server and it definitely didn't run right on php 5.4. I wonder if we want to make 5.6 the recommended minimum.