Skip to content

Feature5#136

Open
pinak22dhir wants to merge 3 commits intoshrey141102:mainfrom
pinak22dhir:feature5
Open

Feature5#136
pinak22dhir wants to merge 3 commits intoshrey141102:mainfrom
pinak22dhir:feature5

Conversation

@pinak22dhir
Copy link

added food ordering system,resolve issue #115

@shrey141102
Copy link
Owner

Please put it inside a folder, and share screenshots

1 similar comment
@shrey141102
Copy link
Owner

Please put it inside a folder, and share screenshots

@shrey141102
Copy link
Owner

Its an empty file?

@pinak22dhir
Copy link
Author

it is a simple project and did not have images of food just a class code, i made the changes now it is visible to you?

@shrey141102
Copy link
Owner

it is a simple project and did not have images of food just a class code, i made the changes now it is visible to you?

can u show input/output?

@shrey141102
Copy link
Owner

changes are visible 👍🏻

@pinak22dhir
Copy link
Author

yes @shrey141102 , it is too visible when you run the code

@shrey141102
Copy link
Owner

yes @shrey141102 , it is too visible when you run the code

I cant run it without merging..
Can you please follow the PR template that i have put?
thank you

@shrey141102
Copy link
Owner

because every other PR is done in the same way

@pinak22dhir
Copy link
Author

sure @shrey141102

@pinak22dhir
Copy link
Author

pinak22dhir commented Jan 9, 2024

Describe your changes :
added food_orderin.py
Issue ticket number and link:
resolves issue #115
Screenshots of your project output, whatever you have added
Uploading Screenshot 2023-09-21 100516.png…

Checklist before requesting a review
I have performed a self-review of my code
If it is a new project, I have done thorough tests.
Have you updated the README file of the repository and added the name of the project in it?
Have you added a README.md file inside the directory of your project and explained it there properly (with images)?

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

Comments