Skip to content

Conversation

@martinzerty
Copy link
Contributor

@martinzerty martinzerty commented Nov 19, 2025

Summary by CodeRabbit

  • Refactor
    • Removed the back navigation link from the compliance assessments table view header, simplifying the navigation interface.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Nov 19, 2025

Walkthrough

Removed the back navigation anchor element from the compliance assessments table mode page header, eliminating the explicit back-navigation UI control and associated left arrow icon without affecting underlying control flow or rendering logic.

Changes

Cohort / File(s) Summary
Back navigation removal
frontend/src/routes/(app)/(third-party)/compliance-assessments/[id=uuid]/table-mode/+page.svelte
Removed the back navigation anchor, left arrow icon, and go-back text from the header

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

  • Straightforward UI element removal with no logic or control flow changes
  • Single file affected with minimal surface area

Suggested reviewers

  • eric-intuitem
  • ab-smith

Poem

A rabbit hops through the interface clear,
Removing the back-arrow path so dear—
No more retracing through UI space,
We forge ahead at a brisker pace! 🐰✨

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: removing a back navigation button from the TPRM table mode interface.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch CA-1326-return-to-audit-useless-in-tprm-mode

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between b8debe7 and f95edfe.

📒 Files selected for processing (1)
  • frontend/src/routes/(app)/(third-party)/compliance-assessments/[id=uuid]/table-mode/+page.svelte (0 hunks)
💤 Files with no reviewable changes (1)
  • frontend/src/routes/(app)/(third-party)/compliance-assessments/[id=uuid]/table-mode/+page.svelte
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (7)
  • GitHub Check: Analyze (python)
  • GitHub Check: enterprise-startup-docker-compose-test
  • GitHub Check: startup-docker-compose-test
  • GitHub Check: enterprise-startup-functional-test (3.12)
  • GitHub Check: startup-functional-test (3.12)
  • GitHub Check: build_enterprise_frontend
  • GitHub Check: build_community_frontend

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.

Copy link
Collaborator

@nas-tabchiche nas-tabchiche left a comment

Choose a reason for hiding this comment

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

This implementation simply removes the "Back to audit" button.
What we effectively want to do here is to only show it to the user if they are not a third-party user.
This can be checked by checking that !page.data.user.is_third_party.

For reference:

{#if !page.data.user.is_third_party}
<Checkbox
{form}
field="is_locked"
label={m.isLocked()}
helpText={m.isLockedHelpText()}
cacheLock={cacheLocks['is_locked']}
bind:cachedValue={formDataCache['is_locked']}
/>
{/if}

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.

3 participants