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

Removes CL from ID Computer & updates "centcom" #3725

Closed

Conversation

realforest2001
Copy link
Member

@realforest2001 realforest2001 commented Jun 25, 2023

About the pull request

People were using the CL's preset in ID computer to gain access to CLs office when this shouldn't really be possible. This stops that, and also makes it so switching the computer to WY mode actually does something.

Explain why it's good for the game

Testing Photographs and Procedure

Screenshots & Videos

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

Changelog

🆑
add: Added proper assignments for wey-yu mode on ID computers.
del: Removed CL from ID Computer unless on WY mode.
/:cl:

@github-actions github-actions bot added Feature Feature coder badge Removal snap labels Jun 25, 2023
@realforest2001 realforest2001 requested a review from fira June 26, 2023 14:08
@github-actions
Copy link
Contributor

github-actions bot commented Jul 4, 2023

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

@github-actions github-actions bot added the Stale beg a maintainer to review your PR label Jul 4, 2023
Copy link
Contributor

@Drulikar Drulikar left a comment

Choose a reason for hiding this comment

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

This all appears working except that the access tab (the first one) is not altered when in wey_yu mode. Namely the grant all button behaves significantly differently in wey_yu mode but the individual regions displayed are still the marine ones.

I don't know exactly what regions should be displayed other than the default, but the grant all access behavior is different than the individual regions when grant all should just be a select all of the individual options button.

In marine mode, grant all grants 34 items, but selecting every entry individually is only 31 items (you can get this up to 35 items if you grant all and then also grant region on Almayer Command because the Commander's quarters doesn't work if you click it - only via region - and it isn't part of grant all).

In wey yu mode, grant all grants 6 items, none of which are the 31-32 you can individually select.

If you try to max access via both modes you can get up to 41 entries.

By default when spawned via create_humans, a CL has this 13 items in their access list:
image

So all of this is to say that the grant all behavior does not line up with the listed regions. In marine mode it is close.

Feel free to undraft and we can move forward with this PR if you would rather address this separately.

@Drulikar Drulikar removed the Stale beg a maintainer to review your PR label Jul 6, 2023
@Drulikar Drulikar marked this pull request as draft July 6, 2023 12:01
@github-actions
Copy link
Contributor

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

@github-actions github-actions bot added the Stale beg a maintainer to review your PR label Jul 14, 2023
@github-actions github-actions bot closed this Jul 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature Feature coder badge Removal snap Stale beg a maintainer to review your PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants