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

new put api added #35

Closed
wants to merge 6 commits into from
Closed

Conversation

GANGSTER0910
Copy link

In this, I have added the put api for the post request of analyze_img_file here first we have to add the data through post api and then with the help of put api we can change the orignal data and we can also see the changes of old and new file #16

@ashitaprasad
Copy link
Member

Can you create a different POST API which is simpler (text bases) and add PUT API for it.

@GANGSTER0910
Copy link
Author

Can you create a different POST API which is simpler (text bases) and add PUT API for it.

I have done the work you have asked, so please review it once more

src/routes/io/io.py Outdated Show resolved Hide resolved
src/routes/io/io.py Outdated Show resolved Hide resolved
src/routes/io/io.py Outdated Show resolved Hide resolved
src/routes/io/io.py Outdated Show resolved Hide resolved
@GANGSTER0910
Copy link
Author

I have done the changes you have asked, sorry for above code and please review it

src/routes/io/io.py Outdated Show resolved Hide resolved
@GANGSTER0910
Copy link
Author

The second file was added by mistake so ignore it, I have also removed the changes from commit

Copy link
Member

Choose a reason for hiding this comment

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

Why have you deleted this file.

@ashitaprasad
Copy link
Member

Merging this change will delete case_convert.py from main repo.
Make a fresh branch .. make only changes in the required file and raise a new PR.

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.

None yet

2 participants