search

ballerina-platform / ballerina-library

The Ballerina Library
https://ballerina.io/learn/api-docs/ballerina/
Apache License 2.0
136 stars 64 forks source link

Introduce a Ballerina connector for Mistral REST API #7046

Open NipunaRanasinghe opened 1 month ago

NipunaRanasinghe commented 1 month ago

We are planning to introduce a new Ballerina connector for the latest Mistral REST API, generated using it's OpenAPI specification.

Related links:

This includes the following tasks:

  1. Create a new repository under 'ballerina-platform' organization.
  2. Review and sanitize the OpenAPI specification.
  3. Generate the connector client.
  4. Implement a test suite covering the core functionalities.
  5. Write documentation with examples.
someshfengde commented 1 month ago

Hi @NipunaRanasinghe can you please assign this issue to me

thanks :)

someshfengde commented 1 month ago

thanks :)

someshfengde commented 1 month ago

Hi @keizer619 don't have access to create a repo under ballerina-platform org?

someshfengde commented 1 month ago

Also can you please provide me an example of existing connector client?

someshfengde commented 1 month ago

@keizer619 is there any update on this

Nuvindu commented 1 month ago

👋 Welcome, @someshfengde ! 🚀

We're thrilled to have you join the Ballerina Hacktoberfest community! The issue has been assigned to you, and we’re excited to see your contributions.

To help you get started, here are a few essential resources:

  1. Connector Contributor Guide: Make sure to go through our Ballerina Hacktoberfest connector contributor guidelines and follow the exact steps, to contribute effectively.
  2. Learn Ballerina: Whether you're new to Ballerina or looking to enhance your skills, check out our official learning resources.
  3. Community & Support: If you have technical questions, feel free to ask on Stack Overflow with the Ballerina tag, or join us on Discord to connect with other community members.

No contribution is too small, and your feedback is always welcome! Don’t hesitate to ask questions, propose new ideas, or report issues.

We are currently in the process of creating a GitHub repository for this connector module and will update you once it’s available. Therefore you don't have to create the repository yourself. In the meantime, please go through the relevant resources and documentation related to the connector.

Happy coding and welcome aboard! 🎉

Nuvindu commented 1 month ago

Also can you please provide me an example of existing connector client?

Repository for the Ballerina Gmail connector : https://github.com/ballerina-platform/module-ballerinax-googleapis.gmail

Gmail Connector in the Ballerina Central: https://central.ballerina.io/ballerinax/googleapis.gmail/4.0.1

AzeemMuzammil commented 1 month ago

Hi @someshfengde,

I'll be mentoring your project, and we have created the repository for your connector project with the initial project template. You can start working in this repository: module-ballerinax-mistral.

Please refer to the contribution process section for guidelines on raising a PR: https://ballerina.io/hacktoberfest/connector-contributor-guide/#contribution-process.

Happy coding! 🎉

AzeemMuzammil commented 1 month ago

Hi @someshfengde, have you had a chance to start working on this issue? Please let me know if you're facing any challenges or need any assistance.

AzeemMuzammil commented 3 weeks ago

As we haven't received any updates from @someshfengde in the past week, we are removing the assignment for now. If you are still interested in working on this issue, please comment below. Meanwhile, this issue is now open for contributions from others.

ayeshLK commented 3 weeks ago

@AzeemMuzammil, @thiunuwan began working on the Ballerina SmartSheet connector, but due to limitations in the OpenAPI tool, we weren’t able to proceed with development. If there isn’t an assignee for the Ballerina Mistral connector, he is available and willing to take it on.

thiunuwan commented 3 weeks ago

@AzeemMuzammil, @thiunuwan began working on the Ballerina SmartSheet connector, but due to limitations in the OpenAPI tool, we weren’t able to proceed with development. If there isn’t an assignee for the Ballerina Mistral connector, he is available and willing to take it on.

Yes,I would like to work on this issue, can you please assign this to me?

ayeshLK commented 3 weeks ago

Greetings from Ballerina team @thiunuwan 👋

You can find the repository for the Ballerina Mistral connector here [1]

To provide context, the deadline for the development is established as one month from the date you were officially assigned to the project. But if you encounter any challenges or unforeseen issues, please feel free to reach out, and we can consider adjusting the deadline accordingly.

If you have any questions or need help with anything, please feel free to reach out. Keep up the awesome work! 🚀

[1] - https://github.com/ballerina-platform/module-ballerinax-mistral

ayeshLK commented 2 weeks ago

Hey @thiunuwan 👋

Just checking in on the progress of the connector development.

Were you able to generate the connector using the official OpenAPI specification [1][2] ?

If you have any concerns regarding the connector development, please don't hesitate to reach out.

Happy Coding 🚀

[1] - https://docs.mistral.ai/api/ [2] - https://github.com/mistralai/platform-docs-public/blob/main/openapi.yaml

ayeshLK commented 2 weeks ago

@thiunuwan 👋

Just checking in on the progress. Were you able to generate the client code using the OpenAPI specification ? If so can you send the pull-request so that I could help you by reviewing it.

thiunuwan commented 2 weeks ago

@thiunuwan 👋

Just checking in on the progress. Were you able to generate the client code using the OpenAPI specification ? If so can you send the pull-request so that I could help you by reviewing it.

Hi @ayeshLK , Sorry for the delayed response—I haven’t been feeling well past few days.I was able to generate the client code. I have to write the necessary tests next, then send a pull request for your review

ayeshLK commented 2 weeks ago

Hey @thiunuwan

Sorry to hear that you were not well. Hope you are feeling well now.

I have to write the necessary tests next, then send a pull request for your review

Yes, that would be great. Please let me know if you need any additional assistance.

thiunuwan commented 1 week ago

Hi @ayeshLK , I have submitted a PR for the client code generation and have also set up the environment for testing. Also, could you please confirm if the sample test I have written is correct? Looking forward to your feedback.

ayeshLK commented 1 week ago

Hi @ayeshLK , I have submitted a PR for the client code generation and have also set up the environment for testing. Also, could you please confirm if the sample test I have written is correct? Looking forward to your feedback.

@thiunuwan great, I will review it ASAP

ayeshLK commented 4 days ago

Hey @thiunuwan 👋

I have added set of review comments to your PR. Can you please check ?

thiunuwan commented 4 days ago

I have added set of review comments to your PR. Can you please check ?

Sure, I’ll check them.

ayeshLK commented 2 days ago

@thiunuwan I have added few more minor comments to your PR. Could you please check and resolve them ?

thiunuwan commented 19 hours ago

@thiunuwan I have added few more minor comments to your PR. Could you please check and resolve them ?

Hi @ayeshLK , I've resolved the minor issues you mentioned. Also, I will add more test cases as you mentioned. ( I just wanted to check whether the way I wrote the test cases is correct, which is why I added only one test case. )