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

[Mod]: Universal Resource Transfer #10241

Closed
ProgrammerFailure opened this issue Oct 8, 2024 · 11 comments · Fixed by #10242
Closed

[Mod]: Universal Resource Transfer #10241

ProgrammerFailure opened this issue Oct 8, 2024 · 11 comments · Fixed by #10242

Comments

@ProgrammerFailure
Copy link
Contributor

ProgrammerFailure commented Oct 8, 2024

Mod Name

Universal Resource Transfer

Author(s)

ProgrammerFailure (me)

License

The unlicense

Homepage link

https://forum.kerbalspaceprogram.com/topic/225761-ksp-18x-universal-resource-transfer-v101/

Download link

https://github.com/ProgrammerFailure/Universal-Resource-Transfer/releases

@ProgrammerFailure
Copy link
Contributor Author

ProgrammerFailure commented Oct 8, 2024

I can change the license if needed, I don't really care about that.
Incompatible with @AniruddhKSP Beamed Power Standalone.
Requires ModuleManager
Description: Plugin that enables transmission of arbitrary resources via beamed power.

@HebaruSan
Copy link
Member

HebaruSan commented Oct 8, 2024

Hey @ProgrammerFailure, thanks for the submission! A couple of things as I'm noticing them...

@ProgrammerFailure
Copy link
Contributor Author

Ah, sorry. Give me a few minutes, I'll fix all those.

@ProgrammerFailure
Copy link
Contributor Author

ProgrammerFailure commented Oct 8, 2024

The repo's version file was just retained from the original mod, I'll double check to see if it's right.

ProgrammerFailure added a commit to ProgrammerFailure/Universal-Resource-Transfer that referenced this issue Oct 8, 2024
@ProgrammerFailure
Copy link
Contributor Author

@HebaruSan , I think the issues have been corrected

@HebaruSan
Copy link
Member

Hi, the v1.0.3 release has no ZIP file to download, just the source code.

I guess this is an opportunity to plug @JonnyOThan's KSP Build Tools, which can automate the whole processs of making a release and more:

https://github.com/KSPModdingLibs/KSPBuildTools

@ProgrammerFailure
Copy link
Contributor Author

Oof I'm an idiot

@ProgrammerFailure
Copy link
Contributor Author

I'm aware of KSP build tools, but I wasn't when I started this mod, hence why it's not used.

1 similar comment
@ProgrammerFailure
Copy link
Contributor Author

I'm aware of KSP build tools, but I wasn't when I started this mod, hence why it's not used.

@ProgrammerFailure
Copy link
Contributor Author

@HebaruSan
Hopefully, it should be fixed now. Sorry for the repeated mistakes.

@HebaruSan
Copy link
Member

HebaruSan commented Oct 8, 2024

Hey @ProgrammerFailure, those changes look good. I noticed a few more minor things, but they don't block merging the PR, so fix or not at your leisure:

  • The website link on the repo links to a comment on page two of the forum thread. I'd suggest removing /page/2/#comment-4414992 from the URL so the user would see the OP upon clicking it.
  • The description on the repo is the same as it was for BeamedPower. You might consider customizing it for your mod.

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

Successfully merging a pull request may close this issue.

2 participants