feat: add ProvidedBy component for resource attribution #692
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add Provider type to ResourceEntry and create ProvidedBy component that displays organization attribution with logo/name support. Supports multiple providers with "+X more" overflow handling.
- Add Share Food Program logo as local asset for testing - Implement image fallback to Avatar when logo fails to load - Increase logo size to 40x40 for better visibility - Position ProvidedBy below Last Modified section
Logo resolution now checks both provider.logo_url and a local mapping by provider name. This allows storing logos as local assets while keeping provider data simple (just name and url).
Component now only displays when resource.providers has actual data.
RNR1
reviewed
Jan 28, 2026
| | 'HIDDEN'; | ||
|
|
||
| /** An organization or entity that provided/contributed this resource. */ | ||
| export type Provider = { |
Contributor
There was a problem hiding this comment.
- When looking in the database, I couldn't find a provider field in the resource table. Is this feature already connected?
- Are we expecting to have more than a single provider per resource? I wonder if having a list of providers per resource makes sense, or if we should connect a single provider per resource.
- I'd expect providers to be a separate table that we join with a foreign key from the resources table when we query for the resource, so that we can define each provider and their logo, url information
Collaborator
Author
There was a problem hiding this comment.
-
nope. We'd have to add a new field for providers.
-
I should probably ask billy about the possiblity of having more than two providers for a resource.
-
I was thinking of hardcoding the logos and the url just because I kinda doubt we'll have many providers, but should probably get clarification 😅.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
Adds "Provided By" feature to display organization attribution when viewing a resource.
Screenshot
Changes
src/types/ResourceEntry.ts- Added Provider typesrc/components/ProvidedBy/ProvidedBy.tsx- New component with extracted ProviderAvatar and ProviderLink subcomponentssrc/utils/providerLogos.ts- Logo mapping utilitysrc/assets/logos/share-food-program.png- Share Food Program logosrc/components/SelectedResourceDetails/SelectedResourceDetails.tsx- IntegrationRefactoring
ProviderAvatarcomponent to handle logo/fallback avatar renderingProviderLinkcomponent for conditional URL wrappingCloses #691
Co-authored-by: Claude [email protected]