-
Notifications
You must be signed in to change notification settings - Fork 156
Pull ARAM item sets from LOLAlytics with LOLFlavor going down #444
base: master
Are you sure you want to change the base?
Conversation
…d translations for LOLAlytics. Also updated a number of packages.
Amusing, I was literally going to make this same change tonight, and thought to glance here and lo and behold, LOL |
Unfortunately, I'm not sure @dustinblackman is still maintaining the app which means it may never make it in. But at least I have it working locally now. |
This actually works, but the imported format from LOLAlytics is really bad compared to good old LOLFlavor. For few champions I've tried there are like all the existing starting items (like 10+ items lol) recommended in the build and all those "most frequent first, second etc. item" just list 5+ different possible items. |
I totally agree. I didn't really like the layout either. I could definitely do some cleanup on it - perhaps reducing the number of items returned might go a long way to improving it. That said, I'd just love to get anything at this point. |
If you're willing to do some cleanup work on it for the greater good, I'll go chat with @dustinblackman. Given my background, and the fact he knows me, I'm hoping I can get him to give me access to collaborate on merging in pull requests and take over maintaining it without having to take the extra step of forking Championify to a new version of my own. (I would much rather leave it living here, since he put so much blood sweat and tears into the project). My vision, since he no longer has the time and resources to maintain it, is to take over maintaining the general releases of it and let the community start contributing more (with adequate review and oversight, naturally!). |
That sounds great @sargonas! I'll carve out some time this week (maybe over next weekend) and see if I can't improve the presentation and usability somewhat and circle back. |
How do I get this pull request on my application? |
Sorry, I've had almost no time the last couple weeks and I've been meaning to get this cleaned up and finished so that it can be integrated into the master branch. If you want something quicker (and have the capability of building it locally) you could fork my repo, build it, and run it locally on your machine. |
@sargonas, Sorry for the delay on getting back to this. I went ahead and cleaned up the look and feel for the LoLAlytics ARAM item sets. I also applied map filters on all LoLAlytics item sets so that the ARAM ones are only on Howling Abyss and the others apply to Summoner's Rift. As part of adding the map filter, I did have to adjust the test fixtures as they were previously set to "map" : "any". Unless you have any other suggestions or adjustments, I think everything is ready to go. If you're able to touch base with @dustinblackman in hopes of getting the app updated I'm sure many would really appreciate it. Thanks a lot! |
I spoke to him briefly back when I checked in, he's keen on the idea of opening up support to others like me to help approve and merge in community updates to keep it alive and fresh, since he no longer has the time to support it, though he says there is some work he wants to do on his side first to make it ready for that, since right now it's not. He didn't explain what exactly, but I suspect it's related to the myriad of dependency and deprecation errors I got myself when I tried to set up the repo from scratch and build my own up to date version. In any event, I did not follow up with him a second time when I should have, so I'll go bug him again shortly. I'm committed one way or another to helping out, just be patient with me while I try. :D |
Understandable. It was quite a struggle getting things working when I pulled the code down the first time (especially on Windows where you can't even run tests). Let him know if he needs any help I, and I'm sure others, would be happy to help clean things up if he just wants to give some direction. Thanks for the help. |
Bumps [handlebars](https://github.com/wycats/handlebars.js) from 4.0.10 to 4.0.14. - [Release notes](https://github.com/wycats/handlebars.js/releases) - [Changelog](https://github.com/handlebars-lang/handlebars.js/blob/v4.0.14/release-notes.md) - [Commits](handlebars-lang/handlebars.js@v4.0.10...v4.0.14) Signed-off-by: dependabot[bot] <[email protected]>
…lebars-4.0.14 Bump handlebars from 4.0.10 to 4.0.14
Bumps [y18n](https://github.com/yargs/y18n) from 3.2.1 to 3.2.2. - [Release notes](https://github.com/yargs/y18n/releases) - [Changelog](https://github.com/yargs/y18n/blob/master/CHANGELOG.md) - [Commits](https://github.com/yargs/y18n/commits) Signed-off-by: dependabot[bot] <[email protected]>
Bumps [ini](https://github.com/isaacs/ini) from 1.3.4 to 1.3.8. - [Release notes](https://github.com/isaacs/ini/releases) - [Commits](npm/ini@v1.3.4...v1.3.8) Signed-off-by: dependabot[bot] <[email protected]>
…-3.2.2 Bump y18n from 3.2.1 to 3.2.2
…1.3.8 Bump ini from 1.3.4 to 1.3.8
Bumps [hosted-git-info](https://github.com/npm/hosted-git-info) from 2.5.0 to 2.8.9. - [Release notes](https://github.com/npm/hosted-git-info/releases) - [Changelog](https://github.com/npm/hosted-git-info/blob/v2.8.9/CHANGELOG.md) - [Commits](npm/hosted-git-info@v2.5.0...v2.8.9) Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [electron](https://github.com/electron/electron) from 1.4.13 to 11.5.0. - [Release notes](https://github.com/electron/electron/releases) - [Changelog](https://github.com/electron/electron/blob/main/docs/breaking-changes.md) - [Commits](electron/electron@v1.4.13...v11.5.0) --- updated-dependencies: - dependency-name: electron dependency-type: direct:development ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [electron](https://github.com/electron/electron) from 11.5.0 to 15.5.5. - [Release notes](https://github.com/electron/electron/releases) - [Changelog](https://github.com/electron/electron/blob/main/docs/breaking-changes.md) - [Commits](electron/electron@v11.5.0...v15.5.5) --- updated-dependencies: - dependency-name: electron dependency-type: direct:development ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [tar](https://github.com/npm/node-tar) from 2.2.1 to 2.2.2. - [Release notes](https://github.com/npm/node-tar/releases) - [Changelog](https://github.com/npm/node-tar/blob/main/CHANGELOG.md) - [Commits](isaacs/node-tar@v2.2.1...v2.2.2) --- updated-dependencies: - dependency-name: tar dependency-type: indirect ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [plist](https://github.com/TooTallNate/node-plist) from 1.1.0 to 3.0.5. - [Release notes](https://github.com/TooTallNate/node-plist/releases) - [Changelog](https://github.com/TooTallNate/plist.js/blob/master/History.md) - [Commits](https://github.com/TooTallNate/node-plist/commits) --- updated-dependencies: - dependency-name: plist dependency-type: direct:development ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [moment](https://github.com/moment/moment) from 2.24.0 to 2.29.2. - [Release notes](https://github.com/moment/moment/releases) - [Changelog](https://github.com/moment/moment/blob/develop/CHANGELOG.md) - [Commits](moment/moment@2.24.0...2.29.2) --- updated-dependencies: - dependency-name: moment dependency-type: direct:production ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [moment](https://github.com/moment/moment) from 2.24.0 to 2.29.2. - [Release notes](https://github.com/moment/moment/releases) - [Changelog](https://github.com/moment/moment/blob/develop/CHANGELOG.md) - [Commits](moment/moment@2.24.0...2.29.2) --- updated-dependencies: - dependency-name: moment dependency-type: direct:production ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Pull ARAM item sets from LOLAlytics with LOLFlavor going down. Updated translations for LOLAlytics. Also updated a number of packages.