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

Don't allow draft shuttles as replacement service #1138

Merged
merged 3 commits into from
Feb 28, 2025

Conversation

lemald
Copy link
Member

@lemald lemald commented Feb 27, 2025

Summary of changes

Asana Ticket: 🏹🐛 [extra] Arrow allows draft shuttles to be selected as replacement service

Unfortunately I still don't see a great way of testing the LiveSelect behavior in unit tests, but I am open to suggestions and I have performed manual testing.

Reviewer Checklist

  • Meets ticket's acceptance criteria
  • Any new or changed functions have typespecs
  • Tests were added for any new functionality (don't just rely on Codecov)
  • This branch was deployed to the staging environment and is currently running with no unexpected increase in warnings, and no errors or crashes.

@lemald lemald marked this pull request as ready for review February 27, 2025 20:17
@lemald lemald requested review from a team and cmaddox5 and removed request for a team February 27, 2025 20:18
Copy link
Contributor

@cmaddox5 cmaddox5 left a comment

Choose a reason for hiding this comment

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

Looks good! Just one small typo and then good to go.

@lemald lemald force-pushed the lem-no-draft-shuttles-as-replacement-service branch from a96e9af to f478aef Compare February 28, 2025 15:25
@lemald lemald force-pushed the lem-no-draft-shuttles-as-replacement-service branch from f478aef to 97b0b13 Compare February 28, 2025 15:40
@lemald lemald merged commit a8c0a32 into master Feb 28, 2025
10 checks passed
@lemald lemald deleted the lem-no-draft-shuttles-as-replacement-service branch February 28, 2025 15:45
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