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

feat: Annotate Outer Void items [skip ci] #3052

Merged
merged 4 commits into from
Jan 2, 2025
Merged

feat: Annotate Outer Void items [skip ci] #3052

merged 4 commits into from
Jan 2, 2025

Conversation

ShadowCat117
Copy link
Member

2024-12-31_17 14 02
2024-12-31_17 14 05

The 2 unique items in the 2nd screenshot are annotated as UnknownGearItem so can't really do anything about that unless we make this annotator a higher priority but I don't think that makes sense.

This is just to get the item highlight on them, don't think there's anything else we can do with them at the moment

@kristofbolyai
Copy link
Member

The 2 unique items in the 2nd screenshot are annotated as UnknownGearItem so can't really do anything about that unless we make this annotator a higher priority but I don't think that makes sense.

Why not? I think the pattern is really strict, so it should not be a problem.

@ShadowCat117
Copy link
Member Author

Why not? I think the pattern is really strict, so it should not be a problem.

I was originally thinking in case they added the tag to gear you get from outer void, but thinking about it again they probably would've done that now and GearAnnotator will catch it once they're added to the API anyway

Copy link
Member

@ryanzhoudev ryanzhoudev left a comment

Choose a reason for hiding this comment

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

I checked against my half page of unique-rare items, seems to work good. Also works as expected for my entries in The Scavenger.

@kristofbolyai kristofbolyai merged commit 1ed0b4d into main Jan 2, 2025
1 check passed
@kristofbolyai kristofbolyai deleted the void-items branch January 2, 2025 14:32
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.

3 participants