Skip to content
This repository has been archived by the owner on Jun 14, 2019. It is now read-only.

provided icons for extensions/api client connector aren't correctly fetched from a CI staged instance #283

Open
gashcrumb opened this issue May 14, 2019 · 3 comments

Comments

@gashcrumb
Copy link
Collaborator

This is a...


[ ] Feature request
[ ] Regression (a behavior that used to work and stopped working in a new release)
[x] Bug report  
[ ] Documentation issue or request

Description

Noticed this after importing an extension that includes it's own icon file. On staging there's an 'openshift' extension now installed that includes a new connector. However the icon file doesn't show up properly:

image

Since it's a connector extension, it also shows up in the create connection wizard:

image

And then as a connection:

image

I think we should consolidate getConnectionIcon, getStepIcon into perhaps a WithIconHelpers component. The trouble is some icons are served out from the UI because they're from built-in steps, and some icons are served out by the backend, using different mechanisms, either by embedding a file, a data URL with an embedded SVG, a db: or extension: URI scheme that requires constructing a URL to the backend in a particular way.

I think basically if we could throw everything at getConnectionIcon anytime we have an object and we need the icon file from it we'd be in good shape. Or maybe to solve this we need to get better at mapping between backend objects and some canonical UI object.

@gashcrumb
Copy link
Collaborator Author

I feel also like we could move all this icon-related stuff out of the api package and into utils or even the syndesis package.

@riccardo-forina
Copy link
Collaborator

They do work when the app is served from the root, so I guess somewhere we are not making use of the PUBLIC_URL thing. We should really clean up the whole thing tho, I agree.

@gashcrumb gashcrumb changed the title provided icons for extensions/api client connector aren't correctly fetched provided icons for extensions/api client connector aren't correctly fetched from a CI staged instance May 23, 2019
@gashcrumb
Copy link
Collaborator Author

Updated the title so it doesn't sound so bad :-)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants