Project Support => General ElkArte discussions => Topic started by: Antechinus on June 24, 2013, 04:31:20 am
Title: Being nice to Spuds and Norv (just for a change).
Post by: Antechinus on June 24, 2013, 04:31:20 am
Ok, I'm making this my official code dump thread. Keep it all in one place. ;) I'll just keep adding zips to this post, and I'll reference them from an update post (including a link to any relevant Github issues). Anyone who wants to can just grab a zip and turn it into a PR. As long as we're going frontwards it's all good. :)
ETA: Deleted the four oldest zips, as they are no longer relevant. ETA2: Removed the next four oldest.
Title: Re: Being nice to Spuds and Norv (just for a change).
Post by: Antechinus on June 24, 2013, 11:10:58 pm
Title: Re: Being nice to Spuds and Norv (just for a change).
Post by: TestMonkey on June 25, 2013, 09:30:16 pm
Indeed, commits per issue to fix is the preferred way, for easier diff, checking out by others, understanding them, investigating history and eventual merge/revert. Thank you for trying to propose updates one at the time.
Re:
Quote from: Antechinus – Also, is anyone going to downlaod the pile of code I did? It's been sitting there for over 48 hours without being touched, but the tiny zip for the editor has already been downloaded.
I'm sure you understand that we may get to attachments posted on forum when we get to them. Don't take it for granted that we'd get to attachments on forum "now". Please remember, zipped batches are outside the normal development process, and they need someone to apply them on a certain point in repo history, to diff and check and commit. I appreciate the assistance Spuds has offered with it, and made it possible for Elk core to get the updates, and split more understandably at that.
Now, updates in small batches makes it easier for everyone. I really appreciate the current thread for this reason too. I hope it's a good compromise. One more bit: could you please specify the commit? (because it's a replace of files, to avoid missing things).
Re: the name of this thread: Sorry, but I won't usually have time to assist (and I don't usually keep tabs on css/theming). I will be clear on this: don't rely on Norv 'cause Norv is caught in reworking the core, breaking Ema's fixes (yes, it's possible!), writing long posts at times, and other must-do's. Grumbling on mooses is an extra. Theming shouldn't wait for me, that'd be a very bad idea (tm). But, people have offered to assist with the necessary, and have done so, thus far. It just won't be "now". It's when we get to them. Someone will (or give an answer), when we can get to it.
That said. I've taken in this round. It's a really cool one IMHO: the cleaning and resorting, and the separation issue by issue, and with copious readme's. With these, you've just upped the standard on good and detailed commits comments! ;D Note taken! (btw, now you realize we expect the same, right? lol)
Thank you for the work on it. Turn this thing around, if that's what you want, it needs it. Just do it so that us mortals can follow it.
For the record, I'll cross-ref here: For a set of fixes proposed here we took into Elk, https://github.com/elkarte/Elkarte/pull/589. For the css refactoring, https://github.com/elkarte/Elkarte/pull/591.
Title: Re: Being nice to Spuds and Norv (just for a change).
Post by: IchBin on June 25, 2013, 09:35:04 pm
Thanks Norv. Was a little confusing for me to find where things were posted and what needed to be put into the code for a PR, since I'm not here every day. I started this topic for ant to post his updates to theme stuff. If he can manage to post any updates to that topic, I'll be glad to do the code merges for him into our repo's.
http://www.elkarte.net/index.php?topic=472.0
Strike that. Somehow I didn't see your OP here Ant. clicked the unread to only see Norv's last post. I'll mark this topic notify and will watch this topic from now on!
Title: Re: Being nice to Spuds and Norv (just for a change).
Post by: IchBin on June 25, 2013, 09:55:48 pm
Maybe someone can add me as moderator so I can change the icon on Ant's posts to indicate I've taken care of the code update? Not sure of the best way to handle the indication, so if you have suggestions I'm all ears. :)
Title: Re: Being nice to Spuds and Norv (just for a change).
Post by: Antechinus on June 25, 2013, 10:00:00 pm
Quote from: Antechinus – Also, is anyone going to download the pile of code I did? It's been sitting there for over 48 hours without being touched, but the tiny zip for the editor has already been downloaded.
I'm sure you understand that we may get to attachments posted on forum when we get to them. Don't take it for granted that we'd get to attachments on forum "now".
Yes I realise that. I was just a tad worried it may have been overlooked by mistake, since the one after it had been grabbed. I wasn't meaning to give anyone a hard time about it.
QuoteNow, updates in small batches makes it easier for everyone. I really appreciate the current thread for this reason too. I hope it's a good compromise. One more bit: could you please specify the commit? (because it's a replace of files, to avoid missing things).
Not sure what you mean here (about specifying the commit).
QuoteRe: the name of this thread: Sorry, but I won't usually have time to assist (and I don't usually keep tabs on css/theming). I will be clear on this: don't rely on Norv 'cause Norv is caught in reworking the core, breaking Ema's fixes (yes, it's possible!), writing long posts at times, and other must-do's. Grumbling on mooses is an extra. Theming shouldn't wait for me, that'd be a very bad idea (tm). But, people have offered to assist with the necessary, and have done so, thus far. It just won't be "now". It's when we get to them. Someone will (or give an answer), when we can get to it.
No prob but, as I understand it, either yourself or Spuds is responsible for the final sign off on merges that are accepted for the main repo, so making my code dumps more comprehensible is still being nice to Norv and Spuds, yes? :D
I'm not saying I expect you to handle all the PR's.
ETA: IchBin as local mod sounds like a plan.
Title: Re: Being nice to Spuds and Norv (just for a change).
Post by: IchBin on June 26, 2013, 08:49:42 pm
Maybe I'll just edit your posts and put a note in that I've taken care of things from here on out.
Title: Re: Being nice to Spuds and Norv (just for a change).
Post by: Antechinus on June 26, 2013, 09:11:32 pm
Hmm, actually that glow tag one could probably do with a slight revision. It's outputting the bbc like this:
Title: Re: Being nice to Spuds and Norv (just for a change).
Post by: Antechinus on July 15, 2013, 07:04:03 pm
Got another one coming up tonight my time. Theme was getting too far away from current repo contents. Sorry about that, but I just kept thinking of stuff while I was doing something else, and thought "Hey might as well...".
Will doc/readme/ it all tonight and bundle it up. 43 files, but many of them are only small changes (deprecated class names, mostly).
Title: Re: Being nice to Spuds and Norv (just for a change).
Post by: Antechinus on July 16, 2013, 08:15:26 am
Title: Re: Being nice to Spuds and Norv (just for a change).
Post by: TE on July 16, 2013, 11:26:35 am
picked that file and wanted to make a PR, but the menu looks horribly broken.. Seems like some of the new language strings are missing... https://github.com/eurich/Elkarte/compare/ant_updates4 $txt['community'] at least..
Title: Re: Being nice to Spuds and Norv (just for a change).
Post by: Antechinus on July 16, 2013, 06:19:51 pm
Ok, yup. I forgot to copy the index.english changes over. It was late. :D
If you just add this one into the mix, it should be fine.
Edit by TE: PR https://github.com/elkarte/Elkarte/pull/689
Title: Re: Being nice to Spuds and Norv (just for a change).
Post by: TE on July 17, 2013, 12:27:10 am