Skip to content
This repository has been archived by the owner on Jul 19, 2023. It is now read-only.

The demo app errors with fresh pod.inrupt.com Pods #21

Closed
Vinnl opened this issue Mar 13, 2021 · 1 comment
Closed

The demo app errors with fresh pod.inrupt.com Pods #21

Vinnl opened this issue Mar 13, 2021 · 1 comment

Comments

@Vinnl
Copy link

Vinnl commented Mar 13, 2021

Search terms you've used

Bug description

The <Profile> component contains

<Image property={VCARD.hasPhoto.iri.value} width={480} /> 

However, new Pods on pod.inrupt.com have no values for vcard:hasPhoto, and the component doesn't properly handle that and thus just throws an error.

(Additionally, the error only says "URL not found for given property" - it would be easier to debug if it had said what the given property was, and that it was looking for an image URL.)

To Reproduce

  1. Follow the "Set up" steps in https://docs.inrupt.com/developer-tools/javascript/react-sdk/application/#set-up
  2. Open https://localhost:3000/
  3. Sign in using https://broker.pod.inrupt.com as a Solid IDP

Expected result

The profile to be shown (without an image).

Actual result

An error is thrown.

Environment

Please run

npx envinfo --system --npmPackages --binaries --npmGlobalPackages --browsers

in your project folder and paste the output here:

$ npx envinfo --system --npmPackages --binaries --npmGlobalPackages --browsers
  System:
    OS: Linux 5.4 Ubuntu 20.04.2 LTS (Focal Fossa)
    CPU: (4) x64 Intel(R) Core(TM) i7-7500U CPU @ 2.70GHz
    Memory: 522.88 MB / 7.51 GB
    Container: Yes
    Shell: 5.8 - /bin/zsh
  Binaries:
    Node: 14.15.3 - ~/.nvm/versions/node/v14.15.3/bin/node
    Yarn: 1.22.5 - /usr/bin/yarn
    npm: 6.14.9 - ~/.nvm/versions/node/v14.15.3/bin/npm
  Browsers:
    Firefox: 86.0
  npmPackages:
    @inrupt/eslint-config-base: 0.0.4 => 0.0.4 
    @inrupt/eslint-config-react: 0.0.4 => 0.0.4 
    @inrupt/lit-generated-vocab-common: ^0.3.11 => 0.3.11 
    @inrupt/solid-client: ^0.4.0 => 0.4.0 
    @inrupt/solid-ui-react: ^1.2.1 => 1.7.0 
    @material-ui/core: ^4.11.0 => 4.11.3 
    @material-ui/icons: ^4.9.1 => 4.11.2 
    @solid/lit-term: ^0.4.1 => 0.4.1 
    @types/node: ^14.10.1 => 14.14.34 
    @types/react: ^16.9.49 => 16.14.5 
    @typescript-eslint/eslint-plugin: ^3.10.1 => 3.10.1 
    @typescript-eslint/parser: ^3.10.1 => 3.10.1 
    eslint: ^7.9.0 => 7.22.0 
    eslint-config-airbnb: ^18.2.0 => 18.2.1 
    eslint-config-prettier: ^6.11.0 => 6.15.0 
    eslint-plugin-import: ^2.22.0 => 2.22.1 
    eslint-plugin-jest: ^23.20.0 => 23.20.0 
    eslint-plugin-jsx-a11y: ^6.3.1 => 6.4.1 
    eslint-plugin-prettier: ^3.1.4 => 3.3.1 
    eslint-plugin-react: ^7.20.6 => 7.22.0 
    eslint-plugin-react-hooks: ^4.1.2 => 4.2.0 
    next: 9.5.2 => 9.5.2 
    prettier: ^2.1.1 => 2.2.1 
    react: 16.13.1 => 16.13.1 
    react-dom: 16.13.1 => 16.13.1 
    typescript: ^4.0.2 => 4.2.3 
  npmGlobalPackages:
    npm: 6.14.9

Additional information

Source: https://forum.solidproject.org/t/react-sample-app-url-not-found-for-given-property/4090

I suppose this might be related to inrupt/solid-ui-react#188?

@Vinnl Vinnl transferred this issue from inrupt/solid-ui-react Mar 13, 2021
@Vinnl
Copy link
Author

Vinnl commented May 10, 2021

Appears to work correctly now!

@Vinnl Vinnl closed this as completed May 10, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant