Clarify share link disable behavior - #132
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
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. Comment |
Greptile SummaryThis change updates share-link wording from revocation to disabling, explains that existing collaborators keep their access, and adds stable controls for share-link automation. The validated access path preserves existing collaborator memberships when a link is disabled and rejects later attempts to redeem that link. The confirmation dialog displays the intended boundary text, and the focused Flutter test and analysis complete successfully. T-Rex validation blocked
Confidence Score: 4/5The verified disable flow protects existing collaborator access and blocks future use of the disabled link. The access boundary was exercised with a source-backed revoke harness, focused Flutter testing, and static analysis. Full confidence is not possible because the application could not render on the web target to directly exercise disabled visual and accessibility controls. Files Needing Attention: No changed file requires a code correction. Resolve the transitive Flutter web compilation issue before relying on browser-level verification of
What T-Rex did
Reviews (2): Last reviewed commit: "Keep share actions accessible through se..." | Re-trigger Greptile |
Summary
The repository and Convex mutation remain named
revokeShareLink; this PR changes only the user-facing contract and dialog automation surface.Verification
fvm flutter test test/share/share_link_copy_test.dartfvm flutter test(320 tests)fvm flutter analyze(only the six pre-existing info notices)fvm flutter build web --no-wasm-dry-run --no-tree-shake-iconsgit diff --checkManual path