Skip to content

Remove Unneccesary Env#188

Merged
jdev082 merged 2 commits intomasterfrom
jdev-rmenv
Mar 14, 2022
Merged

Remove Unneccesary Env#188
jdev082 merged 2 commits intomasterfrom
jdev-rmenv

Conversation

@jdev082
Copy link
Copy Markdown
Collaborator

@jdev082 jdev082 commented Mar 14, 2022

No description provided.

@jdev082 jdev082 changed the title Delete env.js Remove Unneccesary Env Mar 14, 2022
@jdev082 jdev082 merged commit 2b28a5c into master Mar 14, 2022
@jdev082 jdev082 deleted the jdev-rmenv branch March 14, 2022 15:57
@ScolderCreations
Copy link
Copy Markdown
Contributor

well that was quick

Yeah, since you can inject JS into a webview to create constants externally, that env file was unnecessary.

@jdev082
Copy link
Copy Markdown
Collaborator Author

jdev082 commented Mar 14, 2022

well that was quick

Yeah, since you can inject JS into a webview to create constants externally, that env file was unnecessary.

the env.js file had a different purpose then you expected. It contained values such as the PatchID or Patch Level which were both deprecated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants