Skip to content

Fix: Created a CLI login signup system#100

Merged
Once-1296 merged 5 commits into
ProjectX-VJTI:mainfrom
jiyac-code:main
Mar 16, 2026
Merged

Fix: Created a CLI login signup system#100
Once-1296 merged 5 commits into
ProjectX-VJTI:mainfrom
jiyac-code:main

Conversation

@jiyac-code

Copy link
Copy Markdown
Contributor

📝 CoC Python Workshop – Submission PR


👤 Student Details

  • Name: Jiya Choksi
  • GitHub Username: jiyac-code
  • University ID: 251081017
  • Branch / Section: IT

📂 Submission Details

  • Folder Name: jiyac-code_solutions
  • Number of Files Changed/Added: 2

📄 List of Files Changed/Added

  • jiyac-code_solutions/test_playground/projects/CLI_Login/CLI_Login.py
  • jiyac-code_solutions/test_playground/projects/CLI_Login/reg_users.txt

🛠 Description of Changes

I created a CLI login signup system:

  • CLI_Login.py consists of the code which runs the login system.
  • reg_users.txt consists of the names and passwords of the registered users. When a new user is registered, their details are appended to this file. When a registered user tries logging in, the name and password entered are compared with this file.

✅ Self-Declaration

I confirm that:

  • I have followed the repository submission guidelines.
  • I worked on my own fork.
  • I confirm my folder is correctly named
  • All changes are inside my jiyac-code_solutions folder.
  • I have tested my code before submission.
  • This is my own work to the best of my knowledge.

🎯 Thank you for your submission!

@Once-1296

Once-1296 commented Mar 16, 2026

Copy link
Copy Markdown
Collaborator

Great job on a first PR on an issue!👏🏻👌🏻

Slight note tho, altho it valid and I can merge, there can be improvements🤓

  1. Can you hash the password to make it secure?🔒
  2. use clear or cls command with cmd and os in python (i may be wrong on syntax) to clear terminal everytime u switch 'screens'?🤔

I can merge the PR if u wish but if you want to do the improvements, I'll wait too😄, reply and tell me your option!

@jiyac-code

Copy link
Copy Markdown
Contributor Author

Thank you!!
I will do the improvements and commit the updated file.

@Once-1296

Copy link
Copy Markdown
Collaborator

Good job on the changes ! 👍 😄

You have basically completed the logical skeleton of how a simple login/registration system works in many applications under the hood. 🧠

a couple more issues you can see as well as proceed to advanced exercises 🙌🏻

@Once-1296 Once-1296 merged commit 5b77396 into ProjectX-VJTI:main Mar 16, 2026
1 check passed
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