-
-
Notifications
You must be signed in to change notification settings - Fork 160
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
[supersceded by #1381] Implement Course and Resources API #1361
Conversation
@tkurki @sbender9 I have a question regarding saving state on the server. |
I think it is about time to bite the bullet and start using sqlite for this kind of stuff. If not there yet then separate settings json file per "subdomain" / api ? Sqlite3 seems pretty abandoned and does not have prebuilt arm binaries, so takes long enough to compile that people believe install is stuck. There's also https://www.npmjs.com/package/better-sqlite3 but i don't have any experience with that. |
I have not used |
I have used a file I have a number of test curl commands I have used to test both the resources and course APIs.... Should these be located under the |
I think we should have runnable tests - curl commands should be easy enough to turn into sequences of |
A word on naming the persistence directory: I don't think this is about |
I have created the folder |
Following are the commands used for Course API testing: Course Destination position Course Destination href Course RESTART Course ArrivalCircle Course Activate Route pointIndex=0 Course Activate Route pointIndex=0 reverse=true Course Next point Course Previous point Course set pointIndex=3 Clear Destination Clear ActiveRoute |
Following are the commands used for Resources API testing: Using Signal K pathsNew Route Update Route DELETE Route **New Waypoint ** Update Waypoint DELETE Waypoint Create Note Update Note DELETE Note Create Region Update Region Update Region (invalid polygon coordinates) DELETE Region Using API methodsNew Waypoint Update Waypoint INVALID Waypoint New Region New Route INVALID Region |
Have updated the saved state location to be |
Allows typechecking.
shouldAllowPut first Parameter must be the request that is being checked for access.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Submitting what i have so far, see what you make of the comments.
I was thinking about the tests: what if i contribute a sample test that
|
Allows typechecking.
shouldAllowPut first Parameter must be the request that is being checked for access.
…into course-api
supersceded by #1381 |
This PR seeks to implement the Course API outlined in Signal K specification PR #629.
It is a branch of #1352 leveraging the
getResource()
method exposed by theResources
class.