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

reshaping the content (+readme.md update) #13

Merged
merged 1 commit into from
Jan 24, 2023

Conversation

DT-DawidWroblewski
Copy link
Collaborator

fine tuning the content of repos

Copy link
Collaborator

@bigludo7 bigludo7 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks
The link DetailsPage send me a 404 but I guess this is normal as we did not have yet the 2 API documentation. Correct?

@DT-DawidWroblewski DT-DawidWroblewski merged commit ba2b103 into camaraproject:main Jan 24, 2023
@DT-DawidWroblewski
Copy link
Collaborator Author

404 is because I changed the path of md files - I'll update it immediately!

Copy link
Collaborator

@monamok monamok left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry I'm commenting late. I was away during last week. I just put my comments here in order to be more clear and for the changes we can open a new PR.
The idea of having the README file is great but I think with the way the files were moved, we're not aligned with other CAMARA projects anymore. As my understanding, we have to include them like this:
Documentation files --> repository/documentation/xxx.md files (we can still have two subfolders like you did in repository/code/API_definitions). So I think these files (xxx.md files):
https://github.com/camaraproject/NumberVerification/blob/main/code/API_definitions/CAMARA/NUMBER_VERIFICATION_API.md
https://github.com/camaraproject/NumberVerification/blob/main/code/API_definitions/MobileConnect/MC_VERIFIED_MSISDN.md

should be moved back to documentation folder, including or not another subfolder.

I also added some other comments about the resources and versioning.

|Flavour|Endpoint type|Endpoint path|
|---|---|---|
|CAMARA|AuthN/AuthZ||
||resource|/verify|
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/device-phone-number is missing here.


|No|Version|Changelog|
|:---:|---:|:---|
|1|0.5.0|Camara & Mobile Connect flavours available|
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why version 0.5.0? To which document version does this refer?

@monamok monamok mentioned this pull request Jan 25, 2023
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.

3 participants