-
Notifications
You must be signed in to change notification settings - Fork 107
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
[ready] release/3.1.0 #731
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…n load if one exists (#14) - feature/ATC-12 - adds http-server and `start` script to package.json that fires up a local server (#22) - Adds comments to ArrivalBase for future refactor work - Simplifies airline_get method and abstracts addAirline() into its own method. - Abstracts parse logic into helper methods. Updates instances of for/in loops to use forEach instead. - Changes .createChildren() method names to ._init() - Adds doc blocks to new AirportModel methods. - Abstracts if logic out of AirportModel.parse() - Updates getFix to getFixPosition in AirportModel - Updates references
…evation and magnetic_north pulling from this.position. Adds _uniqueId to PositionModel
* Adds stubs for future tests * Begins implementing interface with collections and models to get SID fixes
…rray * TODO: add tests for new classes
* Adds tests for StandardRoutes * Finishes implementation with AirportModel and SidCollection (and children)
…ollection and StandardRouteModel
…. Adds tests to StandardRoutes for new methods.
…be able to handle null/undefined values * Updates test to be more robust and adds test cases for new additions to the models
* Adds test/fixtures folder * Adds getter to PositionModel.magneticNorthInRadians
* Adds documentation to Fix classes
… for a future throw, added comment to if statement stating lack of throw concerns.
bugfix/ATC-115
Feature/ATC-100
…rom zlsa#gh-pages
…TC-96 Conflicts: - src/assets/scripts/airport/AirportModel.js
feature/ATC-96
…iports from zlsa#gh-pages" This reverts commit 5280a88.
…nd re-adds updates to airportLoadList.js
feature/ATC-106
@erikquinn ready to go. that conflict is coming from a deleted file in my fork, and it should be deleted. so if git tries to add it back just say no! be sure to also run |
This was referenced Nov 21, 2016
Closed
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Refactor work, bugfixes and new (js) feature work continues. This branch represents v3.1.0 and will be ready to go 11/20/2016. This branch will include a changelog that will list things that changed since the last version.
currently live on: https://n8rzz.github.io/atc/