Added upper case JPEG extension for icon upload#7200
Added upper case JPEG extension for icon upload#7200DaanHoogland merged 2 commits intoapache:4.18from
Conversation
Codecov Report
@@ Coverage Diff @@
## main #7200 +/- ##
============================================
- Coverage 12.67% 12.67% -0.01%
Complexity 8641 8641
============================================
Files 2716 2716
Lines 256112 256112
Branches 39926 39926
============================================
- Hits 32465 32461 -4
- Misses 219517 219522 +5
+ Partials 4130 4129 -1
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
|
Kudos, SonarCloud Quality Gate passed! |
|
@DaanHoogland a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress. |
|
UI build: ✔️ |
DaanHoogland
left a comment
There was a problem hiding this comment.
tried several mixed case image extension in qa (pNg and jpEg) as well as some regular lower case ones.
|
@jbampton can you approve? |
stephankruggg
left a comment
There was a problem hiding this comment.
CLGTM, not manually tested
|
I manually tested it and it works @harikrishna-patnala @shwstppr can you review and approve |
shwstppr
left a comment
There was a problem hiding this comment.
LGTM
@kiranchavala you may update the PR title to something more suitable
|
Also, can we have it in 4.18 branch? |








Description
This PR updates the file UploadResourceIcon.vue in directory ui/src/components/view
Currently if a file contains Captial JPEG extension, the icon fails to upload
Changed line number 198 to include capital JPEG extension
Types of changes
Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale
Bug Severity
Screenshots (if appropriate):
How Has This Been Tested?