Skip to content

chore: adjust failed response messages#30

Merged
zetsux merged 2 commits intomainfrom
task/adjust-failed-response-message
Nov 3, 2025
Merged

chore: adjust failed response messages#30
zetsux merged 2 commits intomainfrom
task/adjust-failed-response-message

Conversation

@zetsux
Copy link
Owner

@zetsux zetsux commented Nov 3, 2025

Description

This pull request updates user-related message constants in the core/helper/messages/user.go file to use clearer and more concise language. The main focus is on simplifying error and success messages for user operations.

Message improvements:

  • Updated failure messages for user registration, update, delete, login, and picture operations to use more direct and concise wording (e.g., "Failed to register user" instead of "Failed to process user register request"). [1] [2]
  • Improved consistency across all user operation messages, making them easier to read and understand for both developers and users. [1] [2]

Change

Check all options that are relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist

  • I have performed a self-review of my code
  • I have made sure that my fix is effective / my feature works
  • All other features still works locally with my changes
  • I have made API documentation and example response in Postman

@zetsux zetsux self-assigned this Nov 3, 2025
@zetsux zetsux requested a review from Copilot November 3, 2025 12:34
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR simplifies error messages across user-related operations by making them more concise and direct. The changes remove verbose phrasing like "Failed to process user X request" and replace it with shorter "Failed to X user" patterns.

Key Changes:

  • Standardized error message format from "Failed to process user [action] request" to "Failed to [action] user"
  • Applied consistent phrasing across all user operation failures (register, login, update, delete, and picture operations)

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@zetsux zetsux merged commit 6303fa3 into main Nov 3, 2025
1 check passed
@zetsux zetsux deleted the task/adjust-failed-response-message branch November 3, 2025 12:45
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