Problem/Motivation

Awesome to see our project is used here, open source rocks ^^

In the project description you have the following line:

SwisNL MCP Client library (automatically installed via Composer)

I think this should be updated to:

SWIS MCP Client library (swisnl/mcp-client , automatically installed via Composer)

Maybe even just link to our company page (https://www.drupal.org/swis)? We do a lot of Drupal, could be helpfull in showing the dependency is not just a random guy somewhere :)

SWIS MCP Client library (swisnl/mcp-client , automatically installed via Composer)

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Issue fork mcp_client-3560451

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

bbrala created an issue. See original summary.

bbrala’s picture

bbrala
Location Netherlands
commented
Issue summary: View changes
marcus_johansson’s picture

marcus_johansson commented
Status: Active » Needs review

Hi, I added the link and also since you are a Drupal org added you as supporting organizations. I hope that is ok? If you do not want to be there I would remove it.

And thank to you and Joris and team for the framework!

bbrala’s picture

bbrala
Location Netherlands
commented

Awesome thank you. That is fine.

Could you also change the project descrption line in the list to:

SWIS MCP Client library (swisnl/mcp-client , automatically installed via Composer)

We are SWIS not SwisNL ^^

And np prblem, we also use it for our saas product, glad too share with the communiry :)

marcus_johansson’s picture

marcus_johansson commented

Sorry, to much of a developer thinking namspaces is the single source of truth, instead of company names :)

Its updated, I hope its all ok now.

bbrala’s picture

bbrala
Location Netherlands
commented

Awesome thank you.

Laat thing, SWIS is all caps ;)

robertoperuzzo made their first commit to this issue’s fork.

robertoperuzzo opened merge request !8

robertoperuzzo’s picture

robertoperuzzo
Primary language Italian
Location 🇮🇹 Tezze sul Brenta, VI
commented

I also improved the README.md and the documentation. Now SWIS is all caps.

If it is fine, we'll merge it.

bbrala’s picture

bbrala
Location Netherlands
commented
Status: Needs review » Reviewed & tested by the community

Awesome, thank you so much :)

marcus_johansson’s picture

marcus_johansson commented
Status: Reviewed & tested by the community » Fixed

Let's merge an close, thanks all.

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.