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.

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

translate server side error messages. #5505

Merged
merged 7 commits into from
Aug 7, 2024

Conversation

mohsenD98
Copy link
Collaborator

@mohsenD98 mohsenD98 commented Aug 2, 2024

This pull request introduces a new function, which takes a server-side error message and returns a user-friendly version. The function simplifies and clarifies complex error messages, making it easier for users to understand what went wrong and how to fix it.

Key Features:

  • Translates technical error messages into user-friendly messages
  • Provides clear and concise explanations of complex errors

Benefits:

  • Increased customer satisfaction by providing clear explanations of what went wrong
  • Reduced support requests by making it easier for users to troubleshoot issues on their own

Related:
#5493

New Look:

image

image

@mohsenD98 mohsenD98 marked this pull request as draft August 2, 2024 08:31
@mohsenD98 mohsenD98 requested a review from nirvn August 2, 2024 08:31
@mohsenD98 mohsenD98 self-assigned this Aug 2, 2024
src/qml/QFieldCloudPopup.qml Outdated Show resolved Hide resolved
src/qml/QFieldCloudPopup.qml Outdated Show resolved Hide resolved
src/qml/QFieldCloudPopup.qml Outdated Show resolved Hide resolved
@mohsenD98 mohsenD98 marked this pull request as ready for review August 3, 2024 13:13
@nirvn nirvn closed this Aug 3, 2024
@nirvn nirvn reopened this Aug 3, 2024
@qfield-fairy
Copy link
Collaborator

qfield-fairy commented Aug 3, 2024

@nirvn
Copy link
Member

nirvn commented Aug 4, 2024

@suricactus , are there other server error codes you think would be worth making friendlier here?

src/core/utils/qfieldcloudutils.h Outdated Show resolved Hide resolved
src/core/utils/qfieldcloudutils.h Outdated Show resolved Hide resolved
src/core/utils/qfieldcloudutils.h Outdated Show resolved Hide resolved
src/core/utils/qfieldcloudutils.h Outdated Show resolved Hide resolved
src/core/utils/qfieldcloudutils.h Outdated Show resolved Hide resolved
src/core/utils/qfieldcloudutils.cpp Outdated Show resolved Hide resolved
@nirvn nirvn merged commit f763529 into master Aug 7, 2024
22 of 23 checks passed
@nirvn nirvn deleted the translate_server_side_error_messages branch August 7, 2024 02:35
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.

4 participants