-
Notifications
You must be signed in to change notification settings - Fork 4
Redesign organization README for better discoverability #23
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
base: main
Are you sure you want to change the base?
Conversation
Replace the release-table-focused README with a user-persona navigation guide that directs visitors to the right resources based on their intent. Key changes: - Remove 450-line HTML API table (now in API Portfolio) - Add "Where do I start?" navigation for different user types - Link to authoritative sources (API Portfolio, Wiki, Governance) - Add news banner for project announcements Fixes camaraproject/Governance#211
bigludo7
left a comment
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.
Thanks @hdamker
Look good for me - just a small comment for your consideration.
profile/README.md
Outdated
| - **[CAMARA API Design Guide](https://github.com/camaraproject/Commonalities/blob/main/documentation/CAMARA-API-Design-Guide.md)** (Commonalities) | ||
| - **[Common API design and interoperability guidelines](https://github.com/camaraproject/Commonalities)** (Commonalities) | ||
|
|
||
| These documents are maintained by the respective working groups and apply across APIs. |
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.
Very small comment but we haven't document called CAMARA Security and Authentication Profiles and Common API design and interoperability guidelines but instead we target the project git hub.
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.
Maybe for ICM is worth to refer to Security and Interoperability Profile on the one hand and on other hand to Access and User Consent Management
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.
Please review updates in 78988d0
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.
Thanks @hdamker
profile/README.md
Outdated
| Visit our [Wiki](https://lf-camaraproject.atlassian.net/wiki/spaces/CAM/overview?mode=global) for more details about the [Fall25 meta-release](https://lf-camaraproject.atlassian.net/wiki/x/FQApAg) and the [Release Management in CAMARA](https://lf-camaraproject.atlassian.net/wiki/x/US7e). Please note that there are more APIs in work - see the complete list of [Sandbox API Repositories](https://github.com/search?q=topic%3Asandbox-api-repository+org%3Acamaraproject&type=Repositories) which follows also below. | ||
| # CAMARA Project | ||
|
|
||
| CAMARA is a global open-source project that enables the exposure of network capabilities through standardized APIs. The project brings together operators, vendors, and the broader ecosystem to define, evolve, and release interoperable APIs. |
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.
| CAMARA is a global open-source project that enables the exposure of network capabilities through standardized APIs. The project brings together operators, vendors, and the broader ecosystem to define, evolve, and release interoperable APIs. | |
| CAMARA is a global open-source project that enables the exposure of network capabilities and information through standardized APIs. The project brings together network operators, equipment vendors, and the broader ecosystem to define, evolve, and release interoperable APIs. |
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.
@Kevsy To avoid "definition creep" I have reverted the start of the README in 7cf736a back to the previous "Welcome" and pointers to the authoritative project definitions on the website and in the ProjectCharter, (just with the website before the wiki). Here is not the place to refine the project definition ;-)
| --- | ||
|
|
||
| ## Where do I start? |
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.
| ## Where do I start? | |
| ## 📍 Where do I start? |
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.
@Kevsy Adding emojis to every section heading might dilute the effect of 👉 for key call-to-action links (3 times) and make the page busier. Nevertheless the I created the alternative page with the emojis (slightly different selection) to get feedback from the community which version is preferred, see comment below.
|
|
||
| --- | ||
|
|
||
| ### I want to explore CAMARA APIs |
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.
| ### I want to explore CAMARA APIs | |
| ### 🔎 I want to explore CAMARA APIs |
|
|
||
| --- | ||
|
|
||
| ### I want to use or implement a specific API |
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.
| ### I want to use or implement a specific API | |
| ### 🚀 I want to use or implement a specific API |
| If you already know which API you are interested in: | ||
|
|
||
| Start with the **API Portfolio** and select the API (for implementation, use the latest meta-release, currently **Fall25**), then follow the link to the **API repository release** on GitHub, read the repository README and API documentation, and review the published OpenAPI and test specifications. | ||
|
|
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.
If I want to use an API, I will likely be looking for live implementations - so we can add the link to the GSMA map: https://open-gateway.gsma.com/map
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.
Added in 78988d0
|
|
||
| --- | ||
|
|
||
| ### I want to participate in the CAMARA community |
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.
| ### I want to participate in the CAMARA community | |
| ### 👋 I want to participate in the CAMARA community |
|
|
||
| --- | ||
|
|
||
| ### I want to contribute to an API or specification |
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.
| ### I want to contribute to an API or specification | |
| ### 👷 I want to contribute to an API or specification |
|
|
||
| --- | ||
|
|
||
| ### I maintain an API or drive releases |
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.
| ### I maintain an API or drive releases | |
| ### 🏁 I maintain an API or drive releases |
|
|
||
| --- | ||
|
|
||
| ### Looking for something else? |
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.
| ### Looking for something else? | |
| ### 🔭 Looking for something else? |
profile/README.md
Outdated
| - **[CAMARA API Design Guide](https://github.com/camaraproject/Commonalities/blob/main/documentation/CAMARA-API-Design-Guide.md)** (Commonalities) | ||
| - **[Common API design and interoperability guidelines](https://github.com/camaraproject/Commonalities)** (Commonalities) | ||
|
|
||
| These documents are maintained by the respective working groups and apply across APIs. |
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.
Maybe for ICM is worth to refer to Security and Interoperability Profile on the one hand and on other hand to Access and User Consent Management
profile/README.md
Outdated
| - **[CAMARA API Design Guide](https://github.com/camaraproject/Commonalities/blob/main/documentation/CAMARA-API-Design-Guide.md)** (Commonalities) | ||
| - **[Common API design and interoperability guidelines](https://github.com/camaraproject/Commonalities)** (Commonalities) | ||
|
|
||
| These documents are maintained by the respective working groups and apply across APIs. |
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.
Another comment is that in these ICM and Commonalities references we do not indicate that are also related to a certain meta-releases (link to github will go to latest release). Maybe it's good to mention, same as in the portfolio, that the version matching the meta-release of the API to be implemented should be selected
|
|
||
| - **[Contributing Guidelines](https://github.com/camaraproject/Governance/blob/main/CONTRIBUTING.md)** (project-wide contribution process), **[Project Charter](https://github.com/camaraproject/Governance/blob/main/ProjectCharter.md)** and **[Project Structure and Roles](https://github.com/camaraproject/Governance/blob/main/ProjectStructureAndRoles.md)** | ||
| - **[CAMARA API Design Guide](https://github.com/camaraproject/Commonalities/blob/main/documentation/CAMARA-API-Design-Guide.md)** (Commonalities) | ||
| - **[Propose a new API](https://github.com/camaraproject/Governance/blob/main/documentation/API-Onboarding-and-Lifecycle.md)** (API onboarding and lifecycle) |
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.
What about a section for contribution to ICM indicated as applicable if contributing to Security, Identity or Consent topics?
- Fix ICM document references: use actual document names (Security and Interoperability Profile, Access and User Consent Management) - Fix Commonalities references: link to specific documents (API Design Guide, API Testing Guidelines) - Add note about using meta-release-matching document versions - Add GSMA Open Gateway operator map link for live implementations - Add ICM repository to contribution section - Simplify Commonalities reference in contribution section
Address PR review feedback by reverting to the proven upstream-style introduction that points to authoritative sources (Website, Wiki, Governance, Project Charter) instead of defining CAMARA inline. This avoids "definition creep" and keeps the intro concise.
diegogonmar
left a comment
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.
Thanks for addressing my comments @hdamker. LGTM
|
Based on @Kevsy's suggestions I have created an alternative version of the resulting README.md with emojis. Please have a look on the two variants with and without emojis and leave your comment which version you prefer. A compromise as third option could be to keep just 📍 on "Where do I start?" as the main navigation anchor and skip the emojis on subsections. |
|
Preference for the Emoji version 🚀 |
|
I prefer the emoji version |
What type of PR is this?
What this PR does / why we need it:
Replaces the current release-table-focused organization README with a user-persona navigation guide.
Key changes:
Benefits:
Which issue(s) this PR fixes:
Fixes camaraproject/Governance#211
Special notes for reviewers:
Additional documentation
This section can be blank.