Skip to content
This repository has been archived by the owner on Nov 7, 2023. It is now read-only.

Pull ARAM item sets from LOLAlytics with LOLFlavor going down #444

Open
wants to merge 26 commits into
base: master
Choose a base branch
from

Conversation

jeffpignataro
Copy link

Pull ARAM item sets from LOLAlytics with LOLFlavor going down. Updated translations for LOLAlytics. Also updated a number of packages.

Jeff Pignataro added 2 commits April 13, 2019 14:12
@sargonas
Copy link

Amusing, I was literally going to make this same change tonight, and thought to glance here and lo and behold, LOL

@jeffpignataro
Copy link
Author

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.

@LordBoos
Copy link

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.
Don't get this bad, I like that you made this change and we now have at least something, but LOLAlytics format is really bad.

@jeffpignataro
Copy link
Author

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.

@sargonas
Copy link

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!).

@jeffpignataro
Copy link
Author

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.

@oMattz
Copy link

oMattz commented May 4, 2019

How do I get this pull request on my application?

@jeffpignataro
Copy link
Author

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.

@jeffpignataro
Copy link
Author

@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!

@sargonas
Copy link

sargonas commented May 5, 2019

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

@jeffpignataro
Copy link
Author

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.

dependabot bot and others added 7 commits March 30, 2021 02:37
…lebars-4.0.14

Bump handlebars from 4.0.10 to 4.0.14
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]>
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>
dependabot bot and others added 7 commits October 26, 2021 20:08
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>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants