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

update findRandomExitPointForSIDIcao to handle invalid identifier #189

Closed
4 tasks
n8rzz opened this issue Dec 12, 2016 · 7 comments
Closed
4 tasks

update findRandomExitPointForSIDIcao to handle invalid identifier #189

n8rzz opened this issue Dec 12, 2016 · 7 comments

Comments

@n8rzz
Copy link
Owner

n8rzz commented Dec 12, 2016

this all pretains to the StandardRouteCollection.findRandomExitPointForSIDIcao() method

example:

  • airport: KSFO

  • command: ${CALLSIGN} sid offshore9

  • This should error in the ui and the console.

  • there should be a specific check for a valid routeCollection, which throws if one doesn't exist

  • The property name sid should be renamed to routeCollection.

  • add additional test case(s)

note: will be invalidated by zlsa#752

@n8rzz n8rzz added the refactor label Dec 12, 2016
@erikquinn
Copy link
Collaborator

  • ...and please rename to findRandomExitPointforSID().

@n8rzz
Copy link
Owner Author

n8rzz commented Dec 13, 2016

there's that pesky icao sneaking in, good call!

@n8rzz
Copy link
Owner Author

n8rzz commented Dec 13, 2016

@erikquinn actually, will this method even be needed after we implement zlsa#752? Or is that issue more of a 'moonshot' goal? Do you have a timeframe in mind with that issue?

@erikquinn
Copy link
Collaborator

Hm. I guess not. :Smile:

@erikquinn
Copy link
Collaborator

And no that's no moonshot, it's kinda pathetic we don't do it already to be honest. Should be easy. Can be a next spring thing though.

@n8rzz
Copy link
Owner Author

n8rzz commented Jan 13, 2017

like invalidated by #243 and #229

@erikquinn
Copy link
Collaborator

Covered by openscope/openscope#120. Closing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants