Skip to content
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

Match V4 deprecated, needs update #20

Open
ChronoBrake opened this issue Sep 21, 2021 · 11 comments
Open

Match V4 deprecated, needs update #20

ChronoBrake opened this issue Sep 21, 2021 · 11 comments
Assignees
Labels
bug Something isn't working question Further information is requested

Comments

@ChronoBrake
Copy link

Hey Sir,

Match V4 not working anymore, you will update your awesome tool to new version V5?
Regards

@Kla35
Copy link
Owner

Kla35 commented Sep 21, 2021

Hey @ChronoBrake ,

I didn't know my tool was used by people ahah
I can't promise, but I can check this evening !

@Kla35
Copy link
Owner

Kla35 commented Sep 21, 2021

I just look at the new API Match v5, and they make a reworking of the API, so it means that I have to take time to make changes.
Seems like the new API is close to TFT API, so I'm used with it, but I need to recatch all the infos from League Game
Not seems impossible, but I will take me some time while I'm now a full time student
For when do u need it ?

@Kla35
Copy link
Owner

Kla35 commented Sep 21, 2021

@ChronoBrake Hey, I take a moment, and.......... it seems to be more easier than I thought !
I don't know when it will be up, but it won't take a lot of time !

@Kla35
Copy link
Owner

Kla35 commented Sep 21, 2021

Up @ChronoBrake
I'm glad to say that my app is now working with Match V5 ! :D Commit : 91186ba
Seems we have more data than earlier, I'm thinking about adding a MVP picture...
I also ajust dmg chart, to make the two team dmg chart with the biggest dmg in the game

A new change too : Before, when u put the match id, it was only the match id.
Now, u have to put the server where the game was played. I'm gonna see if I can work on this later.
Check example below :
Old match id : 469902960
New match id : **EUW1_**5469902960

/!\ WARNING : As before, this app won't generate postgame on custom game, for the reason that Riot doesn't save data from custom game /!\

EDIT : I've just updated the repo, I didn't update the executable right now, I'll do that tomorrow

@Kla35 Kla35 closed this as completed Sep 21, 2021
@ChronoBrake
Copy link
Author

Okey i i waiting for it <3

@Kla35
Copy link
Owner

Kla35 commented Sep 22, 2021

The 1.2.9 version is now out !
Still a issue in using the username atm, I'm gonna fix it ASAP

@ChronoBrake
Copy link
Author

Yeah it's working also MVP function will be awesome <3

@Kla35
Copy link
Owner

Kla35 commented Sep 23, 2021

@ChronoBrake As I asked before, for what usage do u use this soft ?
I'm just curious about that haha

For the MVP function, I got some ideas to do that, it doesn't seems hard to do, but I have less time now to developp my own software
I can't tell u when it will be integrated, sry D:

@ChronoBrake
Copy link
Author

ChronoBrake commented Oct 13, 2021

Hey latest version giving error when i run it by "npm start". What's the problem? Regards

`> lol-runes-pregame@1.0.0 start
> electron .

node:internal/child_process:415
    throw errnoException(err, 'spawn');
    ^

Error: spawn UNKNOWN
    at ChildProcess.spawn (node:internal/child_process:415:11)
    at Object.spawn (node:child_process:698:9)
    at Object.<anonymous> (C:\Users\yoube\Desktop\lis-generator-master\node_modules\electron\cli.js:7:18)
    at Module._compile (node:internal/modules/cjs/loader:1101:14)
    at Object.Module._extensions..js (node:internal/modules/cjs/loader:1153:10)
    at Module.load (node:internal/modules/cjs/loader:981:32)
    at Function.Module._load (node:internal/modules/cjs/loader:822:12)
    at Function.executeUserEntryPoint [as runMain] (node:internal/modules/run_main:79:12)
    at node:internal/main/run_main_module:17:47 {
  errno: -4094,
  code: 'UNKNOWN',
  syscall: 'spawn'
}`

@ChronoBrake ChronoBrake mentioned this issue Oct 14, 2021
@Kla35
Copy link
Owner

Kla35 commented Oct 14, 2021

I'm pretty surprised 'cause the soft worked for me, even if only sources files.
I've searching a bit on Internet and I find out that the soft won't work because u're not running it from a administrator session, I will check that later

@Kla35 Kla35 reopened this Oct 14, 2021
@Kla35 Kla35 self-assigned this Oct 14, 2021
@Kla35 Kla35 added bug Something isn't working question Further information is requested labels Oct 14, 2021
@ChronoBrake
Copy link
Author

I think the problem is with node_modules and not updated package.json. You can upload your full git version with modules and updated package.json?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants