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.

Sign up for GitHub

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

Jump to bottom

Improve error Messages for ws_makemigration and ws_migrate commands #170

Closed
shahharsh176 opened this issue Mar 28, 2024 · 2 comments · Fixed by #258
Closed

Improve error Messages for ws_makemigration and ws_migrate commands #170

shahharsh176 opened this issue Mar 28, 2024 · 2 comments · Fixed by #258
Assignees
@ChrisMcLernon
Labels
feature New feature request good first issue Good for newcomers

Comments

@shahharsh176
Copy link
Contributor

Description

Improve error Messages for various scenarios in ws_makemigration and ws_migrate commands

Background

Both ws_makemigration and ws_migrate commands require arguments. When invalid values are passed as arguments, clear and informative error messages should be raised to guide users in resolving issues.

Please find the list of scenarios and error messages in the Feature Details sections

Feature Details

Failure Scenarios

1. Invalid app_name

If an invalid app_name is supplied in the ws_makemigration or ws_migrate command, the following error message should be displayed:

The app name '{app_name provided by user}' provided as an argument is invalid. 
Please ensure that you have entered the correct app name and try again.
@shahharsh176 shahharsh176 added feature New feature request good first issue Good for newcomers labels Mar 28, 2024
@AjaySDwivedi1
Copy link
Contributor

Can I work on this?

Dibyanshuaman added a commit to Dibyanshuaman/zelthy3 that referenced this issue Mar 28, 2024
@Dibyanshuaman
fix:Improve error Messages for ws_makemigration and ws_migrate commands
d99435f 
Healthlane-Technologies#170
@shahharsh176 shahharsh176 mentioned this issue Mar 29, 2024
Open
ChrisMcLernon added a commit to ChrisMcLernon/Zango that referenced this issue Jun 21, 2024
@ChrisMcLernon
fix:Healthlane-Technologies#170 error handling message Healthlane-Tec…
5acb82c 
…hnologies#171
@ChrisMcLernon
Copy link
Contributor

#258

@shahharsh176 shahharsh176 linked a pull request Jun 21, 2024 that will close this issue
#252, #173, #170: Few Minor Issues Fixed #258
Merged
kc-diabeat pushed a commit that referenced this issue Jun 24, 2024
@ChrisMcLernon
#252, #173, #170: Few Minor Issues Fixed (#258)
f1abdaf 
* modified:   backend/src/zango/apps/appauth/models.py

* modified:   backend/src/zango/cli/start_project.py

* modified:   backend/src/zango/cli/start_project.py

* modified:   backend/src/zango/cli/start_project.py

* Made corrections

* Fixes

* fixed user credential clears

* fix:#170 error handling message #171

* small adjustment to tenantobj to keep consistent

* Fixes: 6-21-2024

* fixed requested issues

* Update ws_makemigration.py

---------

Co-authored-by: Chris McLernon <[email protected]>
Co-authored-by: chris-mclernon <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ChrisMcLernon ChrisMcLernon

Labels
feature New feature request good first issue Good for newcomers
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

#252, #173, #170: Few Minor Issues Fixed ChrisMcLernon/Zango
3 participants
@shahharsh176 @AjaySDwivedi1 @ChrisMcLernon