Skip to content
This repository has been archived by the owner on Sep 11, 2024. It is now read-only.

Fix image upload preview size #12607

Merged
merged 2 commits into from
Jun 12, 2024
Merged

Conversation

dbkr
Copy link
Member

@dbkr dbkr commented Jun 12, 2024

and add a playwright test for it

Regressed by #12591

Checklist

  • Tests written for new code (and old code if feasible).
  • New or updated public/exported symbols have accurate TSDoc documentation.
  • Linter and other CI checks pass.
  • Sign-off given on the changes (see CONTRIBUTING.md).

@dbkr dbkr added the T-Defect Bugs, crashes, hangs, vulnerabilities, or other reported problems label Jun 12, 2024
@dbkr dbkr marked this pull request as ready for review June 12, 2024 13:30
@dbkr dbkr requested a review from a team as a code owner June 12, 2024 13:30
@dbkr dbkr requested review from t3chguy and robintown June 12, 2024 13:30
@t3chguy
Copy link
Member

t3chguy commented Jun 12, 2024

@dbkr is this fixing a regression? what was it regressed by? links help for history

@dbkr
Copy link
Member Author

dbkr commented Jun 12, 2024

Added, don't think it ever got a bug though.

@dbkr dbkr added this pull request to the merge queue Jun 12, 2024
Merged via the queue into develop with commit 25e4515 Jun 12, 2024
39 checks passed
@dbkr dbkr deleted the dbkr/fix_image_upload_preview_size branch June 12, 2024 22:58
@t3chguy
Copy link
Member

t3chguy commented Jun 13, 2024

@dbkr given the regression is in the RC do we want to backport this change?

@dbkr dbkr added the backport staging Label to automatically backport PR to staging branch label Jun 13, 2024
RiotRobot pushed a commit that referenced this pull request Jun 13, 2024
* Fix image upload preview size

and add a playwright test for it

* Add the actual playwright test

(cherry picked from commit 25e4515)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
backport staging Label to automatically backport PR to staging branch T-Defect Bugs, crashes, hangs, vulnerabilities, or other reported problems
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants