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

[Discussion]: Importing remaining patches from FastTrack fork #10

Closed
4 of 8 tasks
sumghai opened this issue Apr 17, 2020 · 12 comments
Closed
4 of 8 tasks

[Discussion]: Importing remaining patches from FastTrack fork #10

sumghai opened this issue Apr 17, 2020 · 12 comments
Assignees
Labels
Discussion Concerns a discussion topic related to Combat Extended.

Comments

@sumghai
Copy link
Contributor

sumghai commented Apr 17, 2020

The following is a TODO list of compatibility patches that still need to be migrated from the defunct CE: FastTrack Edition fork, and updated to CE 2.0 standards:

Mods whose patch implementations were dubious or otherwise need to be rewritten from the ground up:

  • Rimworld of Magic
  • Star Wars - Factions
    • Patches missed final FT release deadline
  • [Ods] Great War German Uniforms
    • Revamp of various Ods WWII uniform mods

Mods that have not yet been updated for RimWorld 1.1, and whose patches will not be imported:

  • Flommel
  • Spidercamp's Dog Pack
@sumghai sumghai added the Discussion Concerns a discussion topic related to Combat Extended. label Apr 17, 2020
@sumghai
Copy link
Contributor Author

sumghai commented Apr 17, 2020

@zhrocks11 @N7Huntsman - With regards to More Vanilla Turrets, I'd like to hold off on bringing over the fixes for #1162 and #1165, at least until the general CE turret bugs have been fixed.

@sumghai sumghai mentioned this issue Apr 17, 2020
3 tasks
@Juanfrank
Copy link

Juanfrank commented Apr 18, 2020

I don't know if I'm supposed to post this here, but I'm working with the Alpha Animals patch alongside Breadbox2k19. Currently in the playtest change until completely functional before I do a PR to this repo.

@sumghai
Copy link
Contributor Author

sumghai commented Apr 18, 2020

@Juanfrank - I'm more than happy for you and Breadbox2k19 to have a crack at Alpha Animals! To help speed things up for you, here's the current FastTrack implementation for your reference:

CEFT_old_patches_Alpha_Animals.zip

@ghost
Copy link

ghost commented Apr 19, 2020

#13 is done

@Juanfrank Juanfrank mentioned this issue Apr 25, 2020
7 tasks
@N7Huntsman
Copy link
Contributor

I've submitted #93 with a patch for NewRatkinPlus, based on an updated version of my patch. We can discuss desired balance/content adjustments the FT version makes as needed.

@sumghai
Copy link
Contributor Author

sumghai commented May 15, 2020

I've submitted #93 with a patch for NewRatkinPlus, based on an updated version of my patch. We can discuss desired balance/content adjustments the FT version makes as needed.

Nice, thanks!

By the way, have the turret-related issues with CE been resolved? At some point I'd like to revisit the More Vanilla Turret patches.

@Breadbox2k19
Copy link
Contributor

I don’t think those kenshi racial patches need any update whatsoever since they are xml only. Same goes for the warhammer gor patch.

@Breadbox2k19
Copy link
Contributor

Can you post all the old compatibility patches here? Updating it for 1.1 is trivial is it’s xml only.

@sumghai
Copy link
Contributor Author

sumghai commented May 26, 2020

@N7Huntsman has already made a PR for his implementation of NewRatkinPlus, so I am going to defer to his version instead of the FT one.

CEFT_old_patches_Kenshi_races.zip - these should be split up into individual faction/race folders

CEFT_old_patches_Warhammer_Gor_races.zip

I will rewrite the More Vanilla Turret patches from scratch once the turret bugs in CE have been fixed.

As for the other patches, those are so horribly outdated, we're better off redoing them from scratch and not as a FastTrack project.

@sumghai
Copy link
Contributor Author

sumghai commented May 28, 2020

Ratkin patches have been merged into the repo.

@N7Huntsman
Copy link
Contributor

@sumghai While pre-existing issue with turrets not functioning has been resolved, fixing it has caused another issue related to reloading them for some turrets to the surface. There's no current estimate on that particular one being fixed, but I don't believe it is quite as problematic as the previous bug. It'd be up to you whether you think it'll prevent you from working on the MVT patch, but I should think you'd be able to do all typical functionality checks.

@N7Huntsman
Copy link
Contributor

Closed for the time being, since I haven't heard from @sumghai for a while. Will reopen if/when he gets back in touch.

SaltyKarl added a commit to SaltyKarl/CombatExtended that referenced this issue Dec 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Discussion Concerns a discussion topic related to Combat Extended.
Projects
None yet
Development

No branches or pull requests

4 participants