decentraland / sites
96%
master: 95%

LAST BUILD BRANCH: dependabot/npm_and_yarn/postcss-8.5.25
DEFAULT BRANCH: master
Repo Added
Token II953l8u7gnj3xxhaQ39yVZG94B5wmrkL regen
Build 744 Last
Files 424
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

LAST BUILD ON BRANCH 0.49.3
branch: 0.49.3
CHANGE BRANCH
x
Sync Branches

coverage: 96.109% (-0.03%) from 96.138%
30013885375

push

github

web-flow 
fix: warn when editing an approved hangout will send it back for review (#688)

* fix: warn when editing an approved hangout will send it back for review

when an owner edits an already-approved hangout in a way that re-triggers
moderation on the events backend, the edit form now shows an amber warning
up front.

`hasModeratedContentChanged` mirrors the exact submit payload mapping and
returns true only when name / description / image / x / y / server actually
differ — the same fields the events backend re-moderates on. editing only
date, recurrence, email, or community does not warn. the form hook exposes
`requiresModerationReview` (approved and content changed) and `EventForm`
renders a `ReviewWarningAlert`. copy added to all six locales.

covered by helper unit tests across the field matrix and form tests for the
warning shown / hidden states.

* fix: align approved-edit warning with backend re-moderation parity

addresses review parity notes on the moderation warning:

- extend `hasModeratedContentChanged` to the full backend moderated set
  this form can edit — adds the vertical image and the world/location mode
  alongside name/description/image/coords/server (categories aren't editable
  here, so they stay omitted).
- gate `requiresModerationReview` on the backend's actor-can-approve
  exemption via `useAdminPermissions`, so moderators / self-approvers don't
  see a "will send it back for review" warning for an edit the backend will
  keep approved.

the sanitized-description round-trip case (a non-moderated edit re-queuing a
legacy event whose stored description held markup) is resolved on the
backend (events#986), which now compares the sanitized description on both
sides so the form baseline and the backend agree.

4345 of 4855 branches covered (89.5%)

Branch coverage included in aggregate %.

10352 of 10437 relevant lines covered (99.19%)

17.88 hits per line

Relevant lines Covered
10437 RELEVANT LINES 10352 COVERED LINES
17.88 HITS PER LINE
Source Files on 0.49.3

Recent builds

Builds Branch Commit Type Ran Committer Via Coverage
30013885375 0.49.3 fix: warn when editing an approved hangout will send it back for review (#688) * fix: warn when editing an approved hangout will send it back for review when an owner edits an already-approved hangout in a way that re-triggers moderation on the ... push web-flow github
96.11
See All Builds (743)

Badge your Repo: sites

We detected this repo isn’t badged! Grab the embed code to the right, add it to your repo to show off your code coverage, and when the badge is live hit the refresh button to remove this message.
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

Refresh