Skip to content

fix(#3714): type error on linkReferences when editing dRep page#3717

Merged
bosko-m merged 1 commit into
developfrom
3714-blank-screen-and-type-error-on-linkreferences-when-navigating-to-edit-drep-page
Jun 3, 2025
Merged

fix(#3714): type error on linkReferences when editing dRep page#3717
bosko-m merged 1 commit into
developfrom
3714-blank-screen-and-type-error-on-linkreferences-when-navigating-to-edit-drep-page

Conversation

@Ciabas
Copy link
Copy Markdown
Contributor

@Ciabas Ciabas commented Jun 3, 2025

List of changes

  • Fix blank screen and type error on linkReferences when navigating to edit dRep page that has no links

Checklist

  • related issue
  • My changes generate no new warnings
  • My code follows the style guidelines of this project
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the changelog
  • I have added tests that prove my fix is effective or that my feature works

@Ciabas Ciabas requested a review from bosko-m June 3, 2025 07:59
@Ciabas Ciabas requested a review from MSzalowski as a code owner June 3, 2025 07:59
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 3, 2025

✅ All checks succeeded

1 similar comment
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 3, 2025

✅ All checks succeeded

@Ciabas Ciabas requested a review from Ryun1 as a code owner June 3, 2025 08:04
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 3, 2025

✅ All checks succeeded

3 similar comments
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 3, 2025

✅ All checks succeeded

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 3, 2025

✅ All checks succeeded

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 3, 2025

✅ All checks succeeded

@Ciabas Ciabas force-pushed the 3714-blank-screen-and-type-error-on-linkreferences-when-navigating-to-edit-drep-page branch from 5800bb2 to 50f7802 Compare June 3, 2025 08:21
image: data?.image ?? "",
linkReferences: data.linkReferences ?? [getEmptyReference("Link")],
identityReferences: data.identityReferences ?? [
linkReferences: data?.linkReferences ?? [getEmptyReference("Link")],
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Previously there was an optional chaining operator, which we missed.
image

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 3, 2025

✅ All checks succeeded

2 similar comments
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 3, 2025

✅ All checks succeeded

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 3, 2025

✅ All checks succeeded

@Ciabas Ciabas force-pushed the 3714-blank-screen-and-type-error-on-linkreferences-when-navigating-to-edit-drep-page branch from 50f7802 to 15c62a7 Compare June 3, 2025 09:46
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 3, 2025

✅ All checks succeeded

2 similar comments
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 3, 2025

✅ All checks succeeded

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 3, 2025

✅ All checks succeeded

@bosko-m bosko-m merged commit ead46a0 into develop Jun 3, 2025
8 checks passed
@bosko-m bosko-m deleted the 3714-blank-screen-and-type-error-on-linkreferences-when-navigating-to-edit-drep-page branch June 3, 2025 11:12
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.

2 participants