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

EZEE-3455: [Image upload] no validation message for files bigger than 2mb #1702

Merged

Conversation

lucasOsti
Copy link
Contributor

@lucasOsti lucasOsti commented Feb 1, 2021

Question Answer
Tickets https://issues.ibexa.co/browse/EZEE-3455
Bug fix? yes
New feature? no
BC breaks? no
Tests pass? yes
Doc needed? no
License GPL-2.0

Checklist:

  • Coding standards ($ composer fix-cs)
  • Ready for Code Review

@lucasOsti lucasOsti marked this pull request as draft February 1, 2021 13:17
@lucasOsti lucasOsti force-pushed the EZEE-3455-Added-notice-file-size-1-5 branch from db9a653 to ebb9824 Compare February 1, 2021 14:18
@lucasOsti lucasOsti marked this pull request as ready for review February 1, 2021 14:19
@lucasOsti lucasOsti changed the title EZEE-3435: [Image upload] no validation message for files bigger than… EZEE-3455: [Image upload] no validation message for files bigger than 2mb Feb 1, 2021
* @returns {Object}
*/
showFileSizeError() {
const label = this.fieldContainer.querySelector(SELECTOR_FIELD_LABEL).innerHTML;

Copy link
Member

Choose a reason for hiding this comment

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

not needed

}

Copy link
Member

Choose a reason for hiding this comment

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

Suggested change

@@ -80,7 +80,7 @@
}
}

[...document.querySelectorAll(SELECTOR_FIELD)].forEach((fieldContainer) => {
[...document.querySelectorAll(SELECTOR_FIELD) ].forEach((fieldContainer) => {
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
[...document.querySelectorAll(SELECTOR_FIELD) ].forEach((fieldContainer) => {
[...document.querySelectorAll(SELECTOR_FIELD)].forEach((fieldContainer) => {

@lucasOsti lucasOsti requested a review from dew326 February 1, 2021 15:36
@bogusez bogusez self-assigned this Feb 2, 2021
@lserwatka lserwatka merged commit 08ce5f1 into ezsystems:1.5 Feb 2, 2021
@lserwatka
Copy link
Member

Could you merge it up?

@lucasOsti
Copy link
Contributor Author

Could you merge it up?

Done

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
4 participants