Skip to content

feat: Show custom resource icons #4020

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.

Sign up for GitHub

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

Merged
merged 14 commits into from
Sep 13, 2022
Merged

feat: Show custom resource icons #4020

Changes from 1 commit
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
e61d3e9
Update examples templateversion
BrunoQuaresma Sep 12, 2022
c764b21
Updaate backend to accept icon
BrunoQuaresma Sep 12, 2022
f9883c7
Fix entities
BrunoQuaresma Sep 12, 2022
1411f05
Fix test
BrunoQuaresma Sep 12, 2022
fc97c5a
Remove test
BrunoQuaresma Sep 12, 2022
a1ba920
Fix proto and tests
BrunoQuaresma Sep 12, 2022
d608290
Update version
BrunoQuaresma Sep 12, 2022
72bb2f0
Update tfplans
BrunoQuaresma Sep 12, 2022
e8b2d9d
Generate provisioner data
BrunoQuaresma Sep 12, 2022
87d7de7
Update FE to use custom icons
BrunoQuaresma Sep 13, 2022
29f66fb
Merge branch 'main' of github.com:coder/coder into bq/3487
BrunoQuaresma Sep 13, 2022
47bb391
Fix migrations
BrunoQuaresma Sep 13, 2022
e5792fa
Fix eslint error
BrunoQuaresma Sep 13, 2022
086f265
Fix template resources table
BrunoQuaresma Sep 13, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Fix eslint error
  • Loading branch information
@BrunoQuaresma
BrunoQuaresma committed Sep 13, 2022
commit e5792facf21053d36dc1873a5542bdf97e56ffd5
4 changes: 3 additions & 1 deletion site/src/components/Resources/ResourceAvatar.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,9 @@ export const ResourceAvatar: React.FC<ResourceAvatarProps> = ({ resource }) => {
const hasIcon = resource.icon && resource.icon !== ""
const avatarSrc = hasIcon
? resource.icon
: iconPathByResource[resource.type] ?? iconPathByResource["null_resource"]
: // resource.type is dynamic so iconPathByResource[resource.type] can be null
// eslint-disable-next-line @typescript-eslint/no-unnecessary-condition
iconPathByResource[resource.type] ?? iconPathByResource["null_resource"]
const styles = useStyles()

return <Avatar className={styles.resourceAvatar} src={avatarSrc} />
Expand Down