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.

Sign up for GitHub

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

Jump to bottom

Add new csd scripts to tts list #33209

Merged
merged 1 commit into from Feb 19, 2020
Merged

Add new csd scripts to tts list #33209

merged 1 commit into from Feb 19, 2020

Conversation

dmcavoy
Copy link
Contributor

@dmcavoy dmcavoy commented Feb 19, 2020

Copied CSD scripts for 2020 yesterday. This adds constants for them and adds Units 2,3,4, and 6 to the TTS list.

Erin007
Erin007 approved these changes Feb 19, 2020
View reviewed changes
Copy link
Contributor

@Erin007 Erin007 left a comment •
edited

Choose a reason for hiding this comment

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

Why don't we have TTS for all CSD scripts?

@dmcavoy
Copy link
Contributor Author

dmcavoy commented Feb 19, 2020

CSD Units 1 and 5 don't have any programming levels so I think thats why they aren't on the list. Not sure what it saves us @Erin007. Maybe @Hamms knows

@dmcavoy dmcavoy merged commit a7df002 into staging Feb 19, 2020
@dmcavoy dmcavoy deleted the tts-script-copy branch February 19, 2020 19:07
@Hamms
Copy link
Contributor

Hamms commented Feb 20, 2020

I'm afraid I don't have any insight into our reasoning there

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Erin007 Erin007 Erin007 approved these changes

@davidsbailey davidsbailey Awaiting requested review from davidsbailey

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@dmcavoy @Hamms @Erin007