semapps icon indicating copy to clipboard operation
semapps copied to clipboard

[field-components] MultiUrlField - Handle not-url values correctly

Open mguihal opened this issue 1 year ago • 1 comments

Hello,

There is currently a bug inside MultiUrlField when value given is not a valid url (or valid urls array). The URL constructor returns an exception in that case which was not catched, causing global error on the page.

I updated the component to display the not-url value in that case (possible in Archipelago because there is no validation on edit forms)

mguihal avatar Apr 20 '24 21:04 mguihal

@srosset81 you didn't approve this PR. Was it ok for you? Can I merge it?

mguihal avatar Jul 09 '24 20:07 mguihal

Yes sure !

srosset81 avatar Jul 12 '24 17:07 srosset81