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

TI API Rel 0.9.4 new Documentation #241

Merged
merged 4 commits into from
May 29, 2024

Conversation

FabrizioMoggio
Copy link
Collaborator

What type of PR is this?
correction
cleanup
documentation

What this PR does / why we need it:
This PR incorporates the whole documentation in the YAML file deleting the Documentation .md file as described:
#229

There is also some minor changes on the YAML code to be aligned with the CAMARA Guidelines and Release Management

- new Release Notes section added in Documentation, to replace the .md documentation file
- Minor changes:
  - modified camara URL to refer to the Edge Cloud WG
  - OAS version now includes "-wip" extension
  - simplified "Servers" section and included "vwip" as version
@FabrizioMoggio
Copy link
Collaborator Author

MegaLinter is passed.

@FabrizioMoggio
Copy link
Collaborator Author

In the Edge Cloud call (21/05) we decided to mode the .md file in:

https://github.com/camaraproject/EdgeCloud/tree/main/documentation/SupportingDocuments

Currently the .md files are in:

https://github.com/camaraproject/EdgeCloud/tree/main/documentation/SupportingDocuments/API_Documentation

@javierlozallu isn't it better to leave the .md file there adding a note in the readme.md file and maybe modifying the name of the documentation file as: "Traffic_Influence_API_Documentation_Obsolete.md"

@javierlozallu
Copy link
Collaborator

In the Edge Cloud call (21/05) we decided to mode the .md file in:

https://github.com/camaraproject/EdgeCloud/tree/main/documentation/SupportingDocuments

Currently the .md files are in:

https://github.com/camaraproject/EdgeCloud/tree/main/documentation/SupportingDocuments/API_Documentation

@javierlozallu isn't it better to leave the .md file there adding a note in the readme.md file and maybe modifying the name of the documentation file as: "Traffic_Influence_API_Documentation_Obsolete.md"

Yes, didn't remember that .md doc is actually on Supporting Documents so we can keep it there and add the Obsolete sufix.

Now I have a doubt @Kevsy @FabrizioMoggio, this folder https://github.com/camaraproject/EdgeCloud/tree/main/documentation/API_documentation should be the one that contains all the official information about the APIs (User Story, Checklist, Documentation_Obsolete*, etc)?? If is the case we can create a subfolder for each API to avoid mixing APIs documentation.

as discussed here: camaraproject#241 (comment)

Also adding a note on top of the file
@FabrizioMoggio
Copy link
Collaborator Author

Still a MegaLinter error due to:

code/API_definitions/EdgeCloud_LcM.yaml

@FabrizioMoggio
Copy link
Collaborator Author

FabrizioMoggio commented May 29, 2024

is there any objection to merge this PR?

@FabrizioMoggio FabrizioMoggio merged commit 1ca2bfa into camaraproject:main May 29, 2024
1 check failed
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