Skip to content

Conversation

@AltamashShaikh
Copy link
Contributor

Description

Adds missing markReportAsSent method

Issue No

Steps to Replicate the Issue

Checklist

  • [✔] Tested locally or on demo2/demo3?
  • [NA] New test case added/updated?
  • [NA] Are all newly added texts included via translation?
  • [NA] Are text sanitized properly? (Eg use of v-text v/s v-html for vue)
  • [✔] Version bumped?

@AltamashShaikh AltamashShaikh added the Needs Review For pull requests that need a code review. label Oct 10, 2025
@AltamashShaikh AltamashShaikh requested a review from a team October 10, 2025 04:03
@AltamashShaikh
Copy link
Contributor Author

doing a release today, so that a new version gets published finally and we can ask Cloud team to add this plugin next week

Copy link
Contributor

@snake14 snake14 left a comment

Choose a reason for hiding this comment

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

I haven't had a chance to test it, but it looks reasonable to me 👍

@AltamashShaikh AltamashShaikh merged commit a5d8475 into 5.x-dev Oct 10, 2025
8 checks passed
@AltamashShaikh AltamashShaikh deleted the fix-mark-report-sent branch October 10, 2025 06:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Needs Review For pull requests that need a code review.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants