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 infinite loops in setFeatureImageCaption for deleted posts #21081

Merged
merged 6 commits into from
Sep 25, 2024

Conversation

ronaldlangeveld
Copy link
Member

@ronaldlangeveld ronaldlangeveld commented Sep 24, 2024

ref ONC-364

  • Adds a condition to check whether the record is deleted or if deleting is in progress before firing the setFeatureImageCaption.
  • Adds tests. Managed to reproduce the issue using tests.

ref ONC-335

- Adds a condition to check whether the record is deleted or if deleting
  is in progress before firing the `setFeatureImageCaption` function,
which leads to infinite loops.
- Adds tests. Managed to reproduce the issue using tests and then fixed
  it accordingly.
@ronaldlangeveld ronaldlangeveld changed the title 🐛 Fixed featureImageCaption causing infinite loop 🐛 [WIP] featureImageCaption causing infinite loop Sep 24, 2024
- previous destroy method didn't update the state correctly
@ronaldlangeveld ronaldlangeveld changed the title 🐛 [WIP] featureImageCaption causing infinite loop 🐛 Fixed infinite loops in setFeatureImageCaption for deleted posts Sep 25, 2024
@ronaldlangeveld ronaldlangeveld marked this pull request as ready for review September 25, 2024 07:48
@ronaldlangeveld ronaldlangeveld enabled auto-merge (squash) September 25, 2024 07:48
@ronaldlangeveld ronaldlangeveld merged commit 052ef69 into main Sep 25, 2024
35 checks passed
@ronaldlangeveld ronaldlangeveld deleted the fixed-feature-image-loop branch September 25, 2024 08:02
vlavrynovych added a commit to vlavrynovych/Ghost that referenced this pull request Sep 25, 2024
* main: (411 commits)
  Shipped comments-ui@0.18.2
  Fixed various visual details in Comments UI (TryGhost#21121)
  Shipped comments-ui@0.18.1
  Update dependency mysql2 to v3.11.3 [SECURITY]
  Added reply button to comment replies
  Update dependency mysql2 to v3.11.2 [SECURITY]
  Fixed flaky member expertise test
  Update dependency mysql2 to v3.11.1 [SECURITY]
  Increased parallelisation of Nx processes
  Update tiptap monorepo to v2.7.2
  Fixed Article feature image not showing up in the feed
  Fixed requiring passing tests for canary builds
  Updated Nx to v19
  🐛 Fixed infinite loops in `setFeatureImageCaption` for deleted posts (TryGhost#21081)
  Fixed versioning and npm visibility for internal apps
  Update dependency html-validate to v8.24.0
  Implemented lazy loading for Inbox & Activities (TryGhost#21106)
  Added sentry logging to local revisions service for localStorage errors (TryGhost#21078)
  Added posts restore UI (TryGhost#21096)
  Fixed bounce rate on stats page (TryGhost#21097)
  ...

# Conflicts:
#	ghost/i18n/locales/uk/comments.json
#	ghost/i18n/locales/uk/portal.json
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant