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

feat: edit extensions documentation to include outbound/internal calls #1878

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

Conversation

ThisIsDemetrio
Copy link
Contributor

@ThisIsDemetrio ThisIsDemetrio commented Jan 2, 2025

Description

This pull request includes updates on the Manage Extensions page, trimming a little the part dedicated to the detail (too many images, I'd say) and include a new section dedicated to the Outbound Calls and the Internal Calls.

The new updates are mostly based on what is available on the Mia-Platform Console frontend, thus might require some polishing and better explanations on the why some choices have been made and give some use cases.

Pull Request Type

  • Documentation content changes
  • Bugfix / Missing Redirects
  • Docusaurus site code changes

PR Checklist

  • The commit message follows our guidelines included in the CONTRIBUTING.md
  • All tests of Lint, cspell and check-content pass. (How to launch them?)
  • No sensitive content has been committed

Other information

@ThisIsDemetrio ThisIsDemetrio requested a review from a team as a code owner January 2, 2025 17:15

![edit extension destination](./img/extensions/edit-extension-destination.png)
<!-- TODO: Do we have here a link with a comprehensible list of these APIs? -->
It is also possible to configure the extension to communicate with other external APIs, as well with the Mia-Platform supported APIs.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I wish to add a link to a comprehensible list of APIs here. I don't know if there's one yet, any suggestion on how better rephrase this to emphasize the concept of internal APIs would be greatly appreciated.


In the configuration page there will also be an URL next to the _Base URL to contact your APIs_ label. This URL is the one to be used to access the APIs from your endpoint.
<!-- TODO: Are security reasons? Should I be more specific? -->
In fact, for security reasons, the URL to be called is not the destination URL itself, but a different one that is the one that will be used to access the APIs
Copy link
Contributor Author

Choose a reason for hiding this comment

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

How can I rephrase this to better explain the security reasons behind the choice to use a Proxy instead of using the Destination URL itself?

@@ -68,35 +71,110 @@ On this page, you can:

## Extension Detail
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I wonder if it is ok to remove the sections Extension Info, Extension Location and Extension Visibility, given the little amount of information included in these paragraphs, and group them in just one section.

The UI however includes a link to each of the three cards related to the aforementioned sections of the documentation. These card are in the same page.

I wonder if it is better to maintain the previous approach (multiple paragraph) in order to have a more consistent UI, or to keep going with the approach suggested in this PR.

docs/console/company-configuration/extensions.md Outdated Show resolved Hide resolved
docs/console/company-configuration/extensions.md Outdated Show resolved Hide resolved
docs/console/company-configuration/extensions.md Outdated Show resolved Hide resolved
docs/console/company-configuration/extensions.md Outdated Show resolved Hide resolved
docs/console/company-configuration/extensions.md Outdated Show resolved Hide resolved
docs/console/company-configuration/extensions.md Outdated Show resolved Hide resolved
docs/console/company-configuration/extensions.md Outdated Show resolved Hide resolved
docs/console/company-configuration/extensions.md Outdated Show resolved Hide resolved
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