Skip to content

minor change step 1#1194

Merged
pooleycodes merged 1 commit into
mainfrom
content-update-must-check
May 15, 2026
Merged

minor change step 1#1194
pooleycodes merged 1 commit into
mainfrom
content-update-must-check

Conversation

@pooleycodes
Copy link
Copy Markdown
Contributor

@pooleycodes pooleycodes commented May 15, 2026

Description

Please replace this line with a brief description of the changes made.

What type of PR is this? (check all applicable)

  • Refactor
  • Feature
  • Bug Fix
  • Optimization
  • Documentation Update

Related Tickets & Documents

  • Closes #

QA Instructions, Screenshots, Recordings

Before

Before screenshot here

After

After screenshot here

Added/updated tests?

We encourage you to keep the code coverage percentage at 80% and above.

  • Yes
  • No, and this is why: Please replace this line with details on why tests have not been included
  • I need help with writing tests

QA sign off

  • Code has been checked and approved
  • Design has been checked and approved
  • Product and business logic has been checked and proved

[optional] Are there any post-deployment tasks we need to perform?

[optional] Are there any dependencies on other PRs or Work?

Summary by CodeRabbit

Release Notes

  • Updates
    • Refined messaging in the confirmation "What happens next" section to provide clearer guidance on data publication steps.

Review Change Stack

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 15, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 1222254a-866a-4025-a3ff-1010ca233a80

📥 Commits

Reviewing files that changed from the base of the PR and between 920ff24 and c0b6450.

📒 Files selected for processing (1)
  • src/views/check/confirmation.html

Walkthrough

The confirmation page template text for the first step under "What happens next" is refined to provide clearer guidance, changing the wording from "Publish your data" to "Make sure that your data is published on your website" in the requestId conditional branch.

Changes

Confirmation Page Copy Update

Layer / File(s) Summary
Step 1 heading text update
src/views/check/confirmation.html
The heading text for step 1 under "What happens next" in the requestId flow is changed to "Make sure that your data is published on your website", replacing "Publish your data" with more explicit guidance.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

  • digital-land/submit#1070: Both PRs edit src/views/check/confirmation.html to change the "What happens next" step text and headings in the confirmation flow.

Suggested reviewers

  • stevenjmesser
  • paris-dp

Poem

🐰 A whisper refined, a sentence made clear,
From "Publish" to "Make sure"—the message sincere,
One line, one small shift, yet the words now ring true,
The user now knows what they're meant to do! ✨

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch content-update-must-check

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@pooleycodes pooleycodes merged commit 1f339c3 into main May 15, 2026
1 of 4 checks passed
@pooleycodes pooleycodes deleted the content-update-must-check branch May 15, 2026 15:57
@github-actions
Copy link
Copy Markdown

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 66.16% 7112 / 10749
🔵 Statements 66.16% 7112 / 10749
🔵 Functions 63.31% 290 / 458
🔵 Branches 77.83% 969 / 1245
File CoverageNo changed files found.
Generated in workflow #1451 for commit c0b6450 by the Vitest Coverage Report Action

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant