cmss13 icon indicating copy to clipboard operation
cmss13 copied to clipboard

Upp req console, tank and cas printers

Open cuberound opened this issue 1 year ago • 6 comments

About the pull request

spliting the work on HvH gamemode to more PRs for easier rewievs. when making this PR I was told req shuttle would be repalced with new system in a week, it has been few months so I am gona add upp shuttle in future PR

Explain why it's good for the game

more flexible and properly instanciented consoles and its own printers and req for UPP is greate for making events

Screenshots & Videos

Put screenshots and videos here with an empty line between the screenshots and the <details> tags.

Changelog

:cl: add: UPP req console, dropship part fabricato and tank part fabricator /:cl:

cuberound avatar Aug 19 '24 16:08 cuberound

okey I have added the consoles for req, cas and tank, I have not done the part regarding upp req elevators as paulrpg is working on changing them intwo new shuttle code

cuberound avatar Aug 24 '24 14:08 cuberound

This pull request has conflicts, please resolve those before we can evaluate the pull request.

cm13-github avatar Oct 05 '24 12:10 cm13-github

Conflicts have been resolved. A maintainer will review the pull request shortly.

cm13-github avatar Oct 05 '24 19:10 cm13-github

address ci fail (indestructible no longer exists)

wrongly looked at merge conflict when solvng it, I tought that the ndestructible was added not removed

cuberound avatar Oct 06 '24 07:10 cuberound

This pull request has conflicts, please resolve those before we can evaluate the pull request.

cm13-github avatar Oct 07 '24 05:10 cm13-github

Conflicts have been resolved. A maintainer will review the pull request shortly.

cm13-github avatar Oct 07 '24 20:10 cm13-github

This PR has been inactive for long enough to be automatically marked as stale. This means it is at risk of being auto closed in ~ 7 days, please address any outstanding review items and ensure your PR is finished, if these are all true and you are auto-staled anyway, you need to actively ask maintainers if your PR will be merged. Once you have done any of the previous actions then you should request a maintainer remove the stale label on your PR, to reset the stale timer. If you feel no maintainer will respond in that time, you may wish to close this PR youself, while you seek maintainer comment, as you will then be able to reopen the PR yourself

cmss13-ci[bot] avatar Oct 27 '24 00:10 cmss13-ci[bot]

This pull request has conflicts, please resolve those before we can evaluate the pull request.

cm13-github avatar Nov 08 '24 11:11 cm13-github

Conflicts have been resolved. A maintainer will review the pull request shortly.

cm13-github avatar Nov 08 '24 12:11 cm13-github

I've done some testing.

The UPP requisitions system appears to be fully functional. I can order items, the elevator arrives and departs, and the railings and gears will work as the USCM does. Questions on the balance cost of the items is a separate issue but the costs appear to be similar to USCM pricing.

The CAS vendor works properly.

The Tank vendors menu appears to crash whenever I use it. But the Tank vendor even for the USCM is broken since its depreciated for the newer vendors in the hanger, so this is beyond the scope of this PR I think.

Steelpoint avatar Dec 06 '24 14:12 Steelpoint

not really interested in a bespoke supply controller for upp, it'd be better to factionate the existing one

that is what the PR does, or at least aims to. if there are places where it is not the case please inform me

cuberound avatar Dec 19 '24 20:12 cuberound

This pull request has conflicts, please resolve those before we can evaluate the pull request.

cm13-github avatar Jan 01 '25 18:01 cm13-github

Conflicts have been resolved. A maintainer will review the pull request shortly.

cm13-github avatar Jan 02 '25 11:01 cm13-github

This pull request has conflicts, please resolve those before we can evaluate the pull request.

cm13-github avatar Jan 05 '25 16:01 cm13-github

Conflicts have been resolved. A maintainer will review the pull request shortly.

cm13-github avatar Jan 06 '25 11:01 cm13-github