Skip to content

Conversation

haleema-khatun
Copy link
Contributor

#issue add input validation
I have added the input validation for users

Copy link

🚀 Thanks for submitting a Pull Request!

Our team will review it as soon as possible.
Please double-check that your PR follows the guidelines, includes a proper description, and references the related issue number.

If any merge conflicts pop up, kindly resolve them before the PA reviews your PR.

Happy coding! 🌟

@CharithaReddy18 CharithaReddy18 merged commit c4789da into CharithaReddy18:main Sep 20, 2025
1 check passed
@kunalverma2512
Copy link
Contributor

#issue add input validation I have added the input validation for users

Hello
haleema khatun
I wanted to argu about that you have created another ChatInput.jsx again with improper location of it ,
ChatInput was already in symptop checker folder inside components folder
Check it Haleema again and use the file already created in components folder

So that folder structure doesnt messed up and look good

@haleema-khatun
Copy link
Contributor Author

I didn't get this , do I need to delete app.py or fix ChatInput.jsx location
I think I didn't change it's location it is same as previous one.

@kunalverma2512
Copy link
Contributor

I didn't get this , do I need to delete app.py or fix ChatInput.jsx location
I think I didn't change it's location it is same as previous one.

I mean you have created ChatInput.jsx inside components older
Right???

So I am just saying Same ChatInput.jsx was already present in Symptop checker folder which is again inside components folders

So their re two files with same name

Also I am not talking about app.py oe any server folder change

Rather I am emphasizing on Client folder in which you have created a file with same name as inside symptom checker folder

Have a look If not then do communicate me again

@haleema-khatun
Copy link
Contributor Author

haleema-khatun commented Sep 22, 2025

okay , ill check it
But my previous PR is merged , and owner of this repo didn't raise this point

@kunalverma2512
Copy link
Contributor

okay , ill check it But my previous PR is merged , and owner of this repo didn't raise this point

Okay no worry at all as now I am working on some other issue so I will do it

@haleema-khatun
Copy link
Contributor Author

Not a problem I'll delete it , I was clarifying things

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.

3 participants