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

Novice protection: detect novice and respawn zones #145

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

MatthewARoy
Copy link
Collaborator

@MatthewARoy MatthewARoy commented Jun 7, 2019

Pull request summary

Description:

Tracking novice and respawn zones from LOAN

Added:

Stores novice/respawn rooms in memory with utils to access both

Testing checklist:

Tested in prod with rooms selected from https://screepspl.us/data/shard2.novice.json
console.log(`Room is a respawn room: ${'W21S33'} checking ${Cartographer.isRespawnRoom('W21S33')}`); console.log(`Room is a respawn room: ${'W11S13'} checking ${Cartographer.isRespawnRoom('W11S13')}`);
[1:30:01 PM][shard2]Room is a respawn room: W21S33 checking true
[1:30:01 PM][shard2]Room is a respawn room: W11S13 checking false

@ahmedcharles
Copy link
Contributor

Is this still useful given that there is now getRoomStatus?

@bencbartlett
Copy link
Owner

No; I will be adding in updated novice zone restrictions in the near future now that that method is in the API

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.

4 participants