Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

🐛Fixed primary_name crashing Settings history #19550

Merged
merged 1 commit into from
Jan 23, 2024

Conversation

ronaldlangeveld
Copy link
Member

refs https://linear.app/tryghost/issue/PROD-329/history-modal-crashes-the-app

  • Added optional chaining operator to avoid null or undefined crashes when the condition doesn't meet and should rather just move on.
  • It now aligns with the other conditions where chaining is used.

refs https://linear.app/tryghost/issue/PROD-329/history-modal-crashes-the-app

- Added optional chaining operator to avoid null or undefined crashes when the
  condition doesn't meet and should rather just move on.
- It now alligns with the other conditions where chaining is used.
@github-actions github-actions bot added the affects:admin Anything relating to Ghost Admin label Jan 23, 2024
@ronaldlangeveld ronaldlangeveld merged commit db5d2d0 into main Jan 23, 2024
18 checks passed
@ronaldlangeveld ronaldlangeveld deleted the primaryname-crash-history branch January 23, 2024 10:05
ronaldlangeveld added a commit that referenced this pull request Jan 23, 2024
fixes
https://linear.app/tryghost/issue/PROD-329/history-modal-crashes-the-app

- Added optional chaining operator to avoid null or undefined crashes
when the condition doesn't meet and should rather just move on.
- It now aligns with the other conditions where chaining is used.
9larsons pushed a commit that referenced this pull request Jan 24, 2024
fixes
https://linear.app/tryghost/issue/PROD-329/history-modal-crashes-the-app

- Added optional chaining operator to avoid null or undefined crashes
when the condition doesn't meet and should rather just move on.
- It now aligns with the other conditions where chaining is used.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
affects:admin Anything relating to Ghost Admin
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant