Skip to content

Conversation

mohiaror
Copy link
Contributor

Make changes work for multiple fileupload instances present in a custom multifield component.

Q                       A
Fixed Issues? Fixes #1, Fixes #2
Patch: Bug Fix?
Minor: New Feature?
Major: Breaking Change?
Tests Added + Pass? Yes
Documentation Provided Yes (code comments and or markdown)
Any Dependency Changes?
License Apache License, Version 2.0

Make changes work for multiple fileupload instances present in a custom multifield component.
@mohiaror mohiaror requested review from YahorC and LSantha June 20, 2024 15:15
@mohiaror
Copy link
Contributor Author

@YahorC I could not understand the change done in this commit - bfc2bd6

I am not sure if that change is related to remoteReference (which are used for polaris' connected assets usecase). I tried to incorporate that fix in my PR. Can you please check if it is Ok?

Copy link

codecov bot commented Jun 20, 2024

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

Incorporate review comments.
@LSantha
Copy link
Contributor

LSantha commented Jun 25, 2024

@mohiaror , could you please add UI test for this fix?

@YahorC
Copy link
Contributor

YahorC commented Jun 25, 2024

@mohiaror Changes that I applied was regarding page properties image section
So for Image component everything was working fine, but page properties sharing the same logic, but node structure, to which we are referring while trying to fetch image path are different for page properties Image section and just image component
Generally fix is looking fine for me, but could you test please, if it works fine for page properties image tab, to make sure it is working fine https://jira.corp.adobe.com/browse/SITES-21939

Copy link

sonarqubecloud bot commented Jul 1, 2024

Copy link

sonarqubecloud bot commented Sep 1, 2025

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.

5 participants