FIX: CookText may be gone before promise resolves (PR #10097)

Bug introduced in 293467a

GitHub

Hey @CvX :wave:

It looks like cook-text can no longer resolve uploads URLs correctly after this change. Mind taking a look when you can?

Do you have an example that’s broken? upload:// urls are still working in the composer as far as I can tell?

1 Like

It’s affecting queued posts with images. Here’s an example: https://meta.discourse.org/review/1482

To reproduce locally:

  • Add a watched word that requires approval.
  • Create a post containing a watched world and an image.
  • Go to the review queue
1 Like

@romanrizzi Submitted FIX: Short URL resolution in cook-text by CvX · Pull Request #10200 · discourse/discourse · GitHub :bug:

1 Like