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

PHEx Infinity #105

Merged
merged 13 commits into from
Jul 21, 2022
Merged

PHEx Infinity #105

merged 13 commits into from
Jul 21, 2022

Conversation

afkvido
Copy link
Contributor

@afkvido afkvido commented Jul 18, 2022

(crx is signed)
@afkvido afkvido added Priority: Medium Somewhat important. Category: PHEx Prodigy Hacking Extension Official Formal or official post by a maintainer of ProdigyPNP. labels Jul 18, 2022
@afkvido afkvido self-assigned this Jul 18, 2022
@afkvido afkvido linked an issue Jul 18, 2022 that may be closed by this pull request
9 tasks
@afkvido
Copy link
Contributor Author

afkvido commented Jul 18, 2022

Hell yea for c888ea9

@afkvido afkvido added this to the 2.2.0 milestone Jul 18, 2022
PHEx/src/disableIntegrity.js Fixed Show resolved Hide resolved
@Erisfiregamer1
Copy link
Contributor

Erisfiregamer1 commented Jul 18, 2022

Dang, that was fast.
Some things to check:

  • Is MV3 going to break this worse?
  • Does it even WORK??
  • Are we ready for this?

Also promoting to major because HOLY FUCK it's revamping how the gamecode is loaded.

@Erisfiregamer1 Erisfiregamer1 added Priority: High Very important. and removed Priority: Medium Somewhat important. labels Jul 18, 2022
Signed-off-by: Eris <eris@erisws.com>
Signed-off-by: Eris <eris@erisws.com>
@Erisfiregamer1
Copy link
Contributor

Erisfiregamer1 commented Jul 18, 2022

Testing extension, if it actually works I'm going to pull out of draft.
EDIT: Syntax error hold on

Signed-off-by: Eris <eris@erisws.com>
Signed-off-by: Eris <eris@erisws.com>
Signed-off-by: Eris <eris@erisws.com>
@afkvido
Copy link
Contributor Author

afkvido commented Jul 18, 2022

We must remove the Build becasue it does NOT sign the CRX.

Signed-off-by: Eris <eris@erisws.com>
@Erisfiregamer1 Erisfiregamer1 marked this pull request as ready for review July 18, 2022 02:33
Copy link
Contributor

@Erisfiregamer1 Erisfiregamer1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Once you iron out the bug with it only working once after install, ready for merge.

@afkvido
Copy link
Contributor Author

afkvido commented Jul 18, 2022

Once you iron out the bug with it only working once after install, ready for merge.

i think you're requesting changes

@Erisfiregamer1
Copy link
Contributor

nah I'm saying "it's good to go just fix this ONE THING"

@afkvido
Copy link
Contributor Author

afkvido commented Jul 18, 2022

that moment when the one error fucks up the whole thing
wait a sec thats called programming

@afkvido
Copy link
Contributor Author

afkvido commented Jul 18, 2022

Dang, that was fast. Some things to check:

  • Is MV3 going to break this worse?
  • Does it even WORK??
  • Are we ready for this?

Also promoting to major because HOLY FUCK it's revamping how the gamecode is loaded.

Hell yea

Signed-off-by: Eris <eris@erisws.com>
@afkvido
Copy link
Contributor Author

afkvido commented Jul 20, 2022

@Erisfiregamer1 @donaldli2020

Critical-High Priority.
I've already updated the P-NP instances to match 2.2.0 in preparation for this, let's go, come on, let's go, yes, I am rushing yall.

Ok whatever imma just uhh do it myself

@afkvido
Copy link
Contributor Author

afkvido commented Jul 20, 2022

ProdigyPNP/InfiniteZero@79987d4

This will be helpful.

@afkvido
Copy link
Contributor Author

afkvido commented Jul 20, 2022

@Erisfiregamer1 @donaldli2020

How does this look?
I think that this does work from my testing.
Please also test it, because I have paranoia and PTSD from pushing bad updates would like to confirm that this works for everyone.

@afkvido
Copy link
Contributor Author

afkvido commented Jul 20, 2022

Also, please do not merge this yet.

Once y'all have reviewed the PR, I need to sign the CRX.

@afkvido
Copy link
Contributor Author

afkvido commented Jul 20, 2022

Wooo, Project Infinite Zero is almost done!

@Erisfiregamer1
Copy link
Contributor

We've got a CORS error, but that won't hurt. Here's the refrence:
Access to fetch at 'https://pnp5.prodigypnp.repl.co/game.min.js' from origin 'https://math.prodigygame.com' has been blocked by CORS policy: No 'Access-Control-Allow-Origin' header is present on the requested resource. If an opaque response serves your needs, set the request's mode to 'no-cors' to fetch the resource with CORS disabled. (fixed on reload)

Copy link
Contributor

@Erisfiregamer1 Erisfiregamer1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It passed the 3-reload test.
I'm pushing the big green button. HERE GOES FUCKING NOTHING.

@Erisfiregamer1 Erisfiregamer1 merged commit ff34cc7 into master Jul 21, 2022
@Erisfiregamer1 Erisfiregamer1 deleted the infinite-zero branch July 21, 2022 12:31
@afkvido
Copy link
Contributor Author

afkvido commented Jul 21, 2022

We've got a CORS error, but that won't hurt. Here's the refrence: Access to fetch at 'https://pnp5.prodigypnp.repl.co/game.min.js' from origin 'https://math.prodigygame.com' has been blocked by CORS policy: No 'Access-Control-Allow-Origin' header is present on the requested resource. If an opaque response serves your needs, set the request's mode to 'no-cors' to fetch the resource with CORS disabled. (fixed on reload)

If it works on reload, then we should be all good.

@afkvido
Copy link
Contributor Author

afkvido commented Jul 21, 2022

It passed the 3-reload test. I'm pushing the big green button. HERE GOES FUCKING NOTHING.

HELL YEA DAWG

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Category: PHEx Prodigy Hacking Extension Official Formal or official post by a maintainer of ProdigyPNP. Priority: High Very important.
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

Infinite Zero Tracking Issue
3 participants