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

Improved Error Messages #282

Open
wants to merge 7 commits into
base: main
Choose a base branch
from
Open

Improved Error Messages #282

wants to merge 7 commits into from

Conversation

Roaster05
Copy link
Contributor

Description

This draft PR enhances error messages for end users by replacing memory references with custom messages based on error codes and locations. These changes improve the clarity and usability of the error messages.

Key Updates

  • Implemented custom error messages for the project_handler module.
  • Errors now provide detailed and actionable information.
    Examples:
    • ERRO[0000] failed to create project: Project abc already exists
    • ERRO[0001] failed to delete project: Project my_project not found or already deleted
  • Error codes and types are mapped from the endpoint documentation:
    Harbor Go Client Documentation.

Notes

  • Current changes are restricted to the project_handler module.
  • Additional modules will be updated after approval of this approach.

@Roaster05 Roaster05 mentioned this pull request Dec 15, 2024
Open
@Roaster05 Roaster05 marked this pull request as ready for review December 20, 2024 07:07
@Roaster05
Copy link
Contributor Author

@bupd @Vad1mo could you please review this PR, Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@Roaster05