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

Add natDexDoublesTier #9921

Closed
wants to merge 5 commits into from
Closed

Add natDexDoublesTier #9921

wants to merge 5 commits into from

Conversation

smudgerox
Copy link
Contributor

Seems to work fine in testing. Required for client teambuilder support.
image
Needs review just to be sure I didn't mess anything up, but it should be ok.

@KrisXV KrisXV changed the title Support Formats Data Tagging for National Dex Doubles Add natDexDoublesTier Nov 26, 2023
config/formats.ts Outdated Show resolved Hide resolved
data/tags.ts Outdated Show resolved Hide resolved
sim/team-validator.ts Outdated Show resolved Hide resolved
@smudgerox
Copy link
Contributor Author

I had to rebase, but it seems fine still. Does anything else need to be fixed/changed for this to be merged in?
I thought I might also need to add the info about the tier tag to server/chat-commands/info.ts so that it can properly give the ND DOU tier, but it might not be necessary, since there's no ND DOU room.

@KrisXV
Copy link
Member

KrisXV commented Jan 3, 2024

spoke on discord, we're not gonna go through with this for now

@KrisXV KrisXV closed this Jan 3, 2024
@smudgerox smudgerox deleted the nddou-tags branch April 23, 2024 12:52
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.

2 participants