Skip to content

Fix: Handle deleted user login error when loading issues#504

Merged
wx-03 merged 2 commits intoCATcher-org:mainfrom
bipplane:fix-deleted-user
Aug 5, 2025
Merged

Fix: Handle deleted user login error when loading issues#504
wx-03 merged 2 commits intoCATcher-org:mainfrom
bipplane:fix-deleted-user

Conversation

@bipplane
Copy link
Copy Markdown
Contributor

@bipplane bipplane commented Aug 2, 2025

Summary:

Fixes #479
Removes the error "Cannot read properties of null (reading 'login')

Type of change:

  • 🐛 Bug Fix

Changes Made:

  • Added null check to default a deleted user to 'ghost', which is a placeholder name for deleted GitHub accounts (https://github.com/ghost)

Screenshots:d

Proposed Commit Message:

Fix: Handle deleted user login error when loading issues

The error "Cannot read properties of null (reading 'login') is caused
by loading repositories containing issues created by deleted GitHub 
users.

This change adds a fallback to 'ghost', the placeholder for deleted GitHub 
accounts, to prevent the error from showing up during runtime.

Checklist:

  • I have tested my changes thoroughly.
  • I have created tests for any new code files created in this PR or provided a link to a issue/PR that addresses this.
  • I have added or modified code comments to improve code readability where necessary.
  • I have updated the project's documentation as necessary.

@wx-03 wx-03 self-requested a review August 3, 2025 14:18
Copy link
Copy Markdown
Contributor

@wx-03 wx-03 left a comment

Choose a reason for hiding this comment

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

lgtm!

@wx-03 wx-03 merged commit a56cd14 into CATcher-org:main Aug 5, 2025
3 checks passed
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.

Cannot read properties of null (reading 'login') for some repos

2 participants