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

Add endpoint to access user notifications observed by virtual user #21

Open
wants to merge 21 commits into
base: main
Choose a base branch
from

Conversation

BenjaminPelletier
Copy link
Member

This PR continues the work in #19 while shifting focus to the subject of the flight_planning API (virtual user).

Additional changes beyond #19's framework for setting up the user_notifications RESTful resource include:

  • Updating API version to reflect addition of new functionality
  • Updating description of permissions granted by interuss.flight_planning.plan
  • Obtaining notification information from the virtual user rather than directly from the USS backend (since flight_planning is primarily an interface between the test director and USS representative/virtual user -- red shirt - green shirt -- and we want to avoid breaking system encapsulation as much as possible)
  • Removing conflict_count since we separately are able to distinguish between unknown, 0, 1, and more than 1 conflicts and I'm not sure where (what kind of test) we would need to use conflict_count in addition to the conflicts enum (but we can add this back if we can identify a test like that and the requirement(s) it would be verifying)

punamverma and others added 20 commits January 4, 2024 23:35
…ht_id as directed during interuss discussion
Co-authored-by: Michael Barroco <michael@orbitalize.com>
Co-authored-by: Michael Barroco <michael@orbitalize.com>
Co-authored-by: Michael Barroco <michael@orbitalize.com>
Co-authored-by: Michael Barroco <michael@orbitalize.com>
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.

None yet

3 participants