Skip to content

Conversation

@Yadavraushan
Copy link

• Mention that you’ve:
• Added API_DOC.md
• Updated README.md with latest endpoints
• Resolved merge conflicts
• Removed unintended submodule

@vercel
Copy link

vercel bot commented Aug 4, 2025

@Yadavraushan is attempting to deploy a commit to the Openml-stack Team on Vercel.

A member of the Team first needs to authorize it.

@IkkiOcean
Copy link
Member

@Yadavraushan
Screenshot 2025-08-05 at 11 01 30 PM

  1. you removed everything from the readme of repositiory and just added api doc part. that was not your task. you task was to enchance it diagrams and replace old information with new and correct one

  2. api docs doesnt contains every api in the backend including ml-api backend.

@Yadavraushan
Copy link
Author

@Yadavraushan Screenshot 2025-08-05 at 11 01 30 PM

  1. you removed everything from the readme of repositiory and just added api doc part. that was not your task. you task was to enchance it diagrams and replace old information with new and correct one
  2. api docs doesnt contains every api in the backend including ml-api backend.
    @IkkiOcean can we talk on linkedn or something ?? for explain

@IkkiOcean
Copy link
Member

@Yadavraushan api docs doesnt handle all the api in the server, some routes present are non existing. fix that

@Yadavraushan
Copy link
Author

@Yadavraushan api docs doesnt handle all the api in the server, some routes present are non existing. fix that

this is the new updated pr link cz in preivous pr shows error

Yadavraushan#1

@IkkiOcean
Copy link
Member

@Yadavraushan I'm talking about new changes itself. for example this route doesnt exist:

Screenshot 2025-08-12 at 12 08 24 PM

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