Skip to content

Conversation

alcercu
Copy link
Contributor

@alcercu alcercu commented Dec 2, 2024

No description provided.

@alcercu alcercu requested a review from jaybuidl December 2, 2024 18:14
Copy link

netlify bot commented Dec 2, 2024

Deploy Preview for kleros-library-of-justice ready!

Name Link
🔨 Latest commit 6a7dccd
🔍 Latest deploy log https://app.netlify.com/projects/kleros-library-of-justice/deploys/68c2eee26a92a900080b507e
😎 Deploy Preview https://deploy-preview-17--kleros-library-of-justice.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

alcercu and others added 30 commits May 21, 2025 17:28
…ns/checkout-4.2.2

chore(deps): bump actions/checkout from 3.5.2 to 4.2.2
…variance-authority-0.7.1

chore(deps): bump class-variance-authority from 0.7.0 to 0.7.1
…ript-5.7.3

chore(deps-dev): bump typescript from 5.6.3 to 5.7.3
…ntl-3.26.3

chore(deps): bump next-intl from 3.25.1 to 3.26.3
…ub/codeql-action-3.28.9

chore(deps): bump github/codeql-action from 2.22.0 to 3.28.9
…node-22.13.4

chore(deps-dev): bump @types/node from 20.17.6 to 22.13.4
…/kleros-sdk-2.2.0

chore(deps): bump @kleros/kleros-sdk from 2.1.10 to 2.2.0
…-security/harden-runner-2.11.0

chore(deps): bump step-security/harden-runner from 2.6.0 to 2.11.0
….23.2

chore(deps): bump viem from 2.21.53 to 2.23.2
…-react-0.475.0

chore(deps): bump lucide-react from 0.460.0 to 0.475.0
The Attachment link in the Evidence component is now rendered only if evidence.fileURI has a value. This prevents a broken link from showing when no attachment is available.
…ength.

Here's what I did:
- I made the Attachment link render conditionally based on whether it's a valid IPFS path (meaning it starts with '/ipfs/').
- I also refactored conditions into constants to make sure everything complies with the 80-character line limit.
Conditionally render Attachment link in Evidence component
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants