-
Notifications
You must be signed in to change notification settings - Fork 57
Update __repr__() method models.py #47
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
Conversation
I chose this additional piece of code (which.. "makes an attempt to return a string that would yield an object with the same value when passed to eval()") because models and test_crud are important in order to have the testing phase of this application work correctly. In the testing phase it is important to have access to printable versions of objects to see and determine what they contain. It is important to be able to easily access what an object contains easily via printing so that at any point of the complex process of testing, the developer can refer to and "evaluate" any object in the correct way.
@pratik-mahalle
pratik-mahalle
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @consarnproject, Thanks for raising this pr.
Unfortunately, this PR doesn’t include any meaningful changes, typos and breaking links are supposed or just content changes are to be reported not fixed. You can do so on the specific two repositories that we provided. In the future, please ensure that a PR in this repo is raised only if you’ve made substantial updates or solved a particular issue. This helps keep the process efficient and organized.
This being a part of an assignment in api fellowship would be counted since you were supposed to create a pr but wont be merged since this is not that relevant of a change. I’m sure you understand, and I appreciate your cooperation on this. We will be closing this PR for now.
consarnproject
commented
Jun 30, 2025
via email
pratik-mahalle
commented
Aug 12, 2025
Yeah, go ahead @consarnproject
I chose this additional piece of code (which.. "makes an attempt to return a string that would yield an object with the same value when passed to eval()") because models and test_crud are important in order to have the testing phase of this application work correctly. In the testing phase it is important to have access to printable versions of objects to see and determine what they contain. It is important to be able to easily access what an object contains easily via printing so that at any point of the complex process of testing, the developer can refer to and "evaluate" any object in the correct way.