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

fix(file upload): change error text of uploaded file from byte to mb #1185

Conversation

Usmanfee
Copy link
Contributor

@Usmanfee Usmanfee commented Sep 30, 2024

Description

converted file size from Bytes to Mega-Byte of uploaded document for better UX.
Screenshot 2024-09-30 at 17 13 39

Changelog entry:

- **File Upload**:
  - changed error text of uploaded file from Byte to MB [#866](https://github.com/eclipse-tractusx/portal-frontend/pull/1185)

Why

It is hard for user to understand long digits in Bytes and easier to read in mega byte size for better understanding and usually the size of the file either comes in KB or MB.

Issue

#1184

Checklist

Please delete options that are not relevant.

  • I have performed a self-review of my own code
  • I have successfully tested my changes locally

CHANGELOG.md Outdated Show resolved Hide resolved
@JordanGerada3 JordanGerada3 self-requested a review October 1, 2024 10:25
Copy link
Contributor

@JordanGerada3 JordanGerada3 left a comment

Choose a reason for hiding this comment

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

LGTM!

Copy link
Contributor

@shubhamv-ss shubhamv-ss left a comment

Choose a reason for hiding this comment

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

lgtm

@Usmanfee Usmanfee marked this pull request as ready for review October 1, 2024 11:10
@Usmanfee Usmanfee requested a review from oyo October 1, 2024 11:11
@Usmanfee Usmanfee requested a review from evegufy October 9, 2024 12:42
CHANGELOG.md Outdated Show resolved Hide resolved
@evegufy evegufy changed the title fix(UI): uploaded document error message fix(file upload): change error text of uploaded file from byte to mb Oct 9, 2024
@evegufy
Copy link
Contributor

evegufy commented Oct 9, 2024

@oyo could you please review?

@evegufy evegufy added this to the Release 24.12 milestone Oct 9, 2024
@Usmanfee Usmanfee force-pushed the fix/upload-document-error-message branch from 3445f89 to da1271c Compare October 9, 2024 15:16
evegufy
evegufy previously approved these changes Oct 9, 2024
oyo
oyo previously requested changes Oct 16, 2024
Copy link
Contributor

@oyo oyo left a comment

Choose a reason for hiding this comment

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

okay but one message hard coded in english

src/components/shared/basic/Dropzone/index.tsx Outdated Show resolved Hide resolved
@Usmanfee
Copy link
Contributor Author

@oyo I have addressed the feedback. Can you please review now ? Thanks

CHANGELOG.md Outdated Show resolved Hide resolved
@Usmanfee Usmanfee force-pushed the fix/upload-document-error-message branch from 4ece1cd to 5a353eb Compare October 24, 2024 13:34
@Usmanfee Usmanfee requested a review from oyo October 24, 2024 13:35
@Usmanfee Usmanfee requested a review from evegufy October 24, 2024 13:35
@evegufy evegufy changed the base branch from main to release/v2.3.0-RC2 October 24, 2024 18:37
@evegufy evegufy changed the base branch from release/v2.3.0-RC2 to release/v2.3.0-RC3 November 4, 2024 19:10
Copy link

sonarcloud bot commented Nov 7, 2024

@evegufy evegufy dismissed oyo’s stale review November 7, 2024 10:16

discussed with oyo/Martin beforehand

@evegufy evegufy merged commit bc1179e into eclipse-tractusx:release/v2.3.0-RC3 Nov 7, 2024
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: USER READY
Development

Successfully merging this pull request may close these issues.

5 participants