Uploaded Images to New S3 Bucket & made changes across TMSDocs#129
Merged
Conversation
📝 WalkthroughWalkthroughThis pull request updates image asset URLs across 27 files by migrating references from Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~5 minutes Possibly related PRs
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
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 |
Nandhini4501
approved these changes
Apr 20, 2026
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (2)
src/pages/docs/test-management/integrations/jira-two-way-integration/manage-test-runs-in-jira.md (1)
50-106:⚠️ Potential issue | 🔴 CriticalImage URLs are inaccessible (HTTP 403) and must be fixed before merging.
All 8 image URLs in this documentation file are returning 403 Forbidden errors, meaning they cannot be displayed. The migration to
website-static-docs.testsigma.comhas broken all image references in this section. Verify that the correct bucket/domain is being used and that S3 permissions are properly configured.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@src/pages/docs/test-management/integrations/jira-two-way-integration/manage-test-runs-in-jira.md` around lines 50 - 106, The eight markdown image links in the Manage Test Runs in Jira doc (e.g., filenames like Connect_Testsigma_to_Issue.png, Linked_Test_Runs_In_Jira.png, Status_Dropdown_in_Jira.png, Add_Defect_from_Jira_TMS.png, Add_defect_Button_Jira.png, Link_Defect_from_Jira.png, Link_defect_dialog.png, Link_defect_Button.png) are returning 403; fix by updating each image reference to a reachable source: either replace the S3 hostname with the correct bucket/domain, switch to the project’s local/static asset path, or use properly signed/public URLs, and verify S3 object permissions (public-read or correct bucket policy) so the images load; after updating, preview the doc to confirm all eight images render.src/pages/docs/test-management/atto/sprint-planner/manage-sprints.md (1)
76-190:⚠️ Potential issue | 🔴 CriticalImage URLs are broken in the "Synchronization of Jira Actions" section—documentation will have missing images.
7 out of 11 image references in this file are inaccessible (returning 403 Forbidden errors). While 4 images with the
Updated_Doc_Images/path are accessible, all images in the "Synchronization of Jira Actions" section using theAtto_Testsigma/path (Sprint_Activate_from_Jira.png, Story_Modified_in_Jira.png, Refine_With_AI.png, Sprint_Closed_in_Jira.png, Deleted_Jira_Sprint.png, Story_Moved_Jira.png, Story_Deleted_Status.png) fail to load. Either the assets need to be uploaded to the correct CDN location or the URLs in the documentation must be corrected before merging.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@src/pages/docs/test-management/atto/sprint-planner/manage-sprints.md` around lines 76 - 190, The "Synchronization of Jira Actions" section contains seven broken image URLs (files: Sprint_Activate_from_Jira.png, Story_Modified_in_Jira.png, Refine_With_AI.png, Sprint_Closed_in_Jira.png, Deleted_Jira_Sprint.png, Story_Moved_Jira.png, Story_Deleted_Status.png) which return 403; fix by either uploading these assets to the CDN under the correct path (Atto_Testsigma/) or updating the markdown image references to point to the accessible Updated_Doc_Images/ variants (or correct CDN paths), then verify each image loads in the rendered page before approving the merge.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@src/pages/docs/test-management/atto/best-practices/generating-test-cases.md`:
- Around line 180-193: The three image links in
src/pages/docs/test-management/atto/best-practices/generating-test-cases.md (the
markdown image references to Refine_Tests_with_AI.png, All_Inputs_for_AI.png,
and Clear_Prompt_Example.png) are returning HTTP 403; replace or fix them so
they serve with HTTP 200: verify the images exist in the S3 bucket or CDN and
correct their public path/ACL, or copy the images into the repo's static assets
and update the three markdown image URLs to the repo-relative paths; after
updating, verify each URL returns 200 and update the PR accordingly.
---
Outside diff comments:
In `@src/pages/docs/test-management/atto/sprint-planner/manage-sprints.md`:
- Around line 76-190: The "Synchronization of Jira Actions" section contains
seven broken image URLs (files: Sprint_Activate_from_Jira.png,
Story_Modified_in_Jira.png, Refine_With_AI.png, Sprint_Closed_in_Jira.png,
Deleted_Jira_Sprint.png, Story_Moved_Jira.png, Story_Deleted_Status.png) which
return 403; fix by either uploading these assets to the CDN under the correct
path (Atto_Testsigma/) or updating the markdown image references to point to the
accessible Updated_Doc_Images/ variants (or correct CDN paths), then verify each
image loads in the rendered page before approving the merge.
In
`@src/pages/docs/test-management/integrations/jira-two-way-integration/manage-test-runs-in-jira.md`:
- Around line 50-106: The eight markdown image links in the Manage Test Runs in
Jira doc (e.g., filenames like Connect_Testsigma_to_Issue.png,
Linked_Test_Runs_In_Jira.png, Status_Dropdown_in_Jira.png,
Add_Defect_from_Jira_TMS.png, Add_defect_Button_Jira.png,
Link_Defect_from_Jira.png, Link_defect_dialog.png, Link_defect_Button.png) are
returning 403; fix by updating each image reference to a reachable source:
either replace the S3 hostname with the correct bucket/domain, switch to the
project’s local/static asset path, or use properly signed/public URLs, and
verify S3 object permissions (public-read or correct bucket policy) so the
images load; after updating, preview the doc to confirm all eight images render.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: a4023c46-3a8c-4df9-b660-2ab2e089e0ef
📒 Files selected for processing (23)
src/components/LeftNav.jsxsrc/pages/docs/test-management/atto/best-practices/generating-test-cases.mdsrc/pages/docs/test-management/atto/sprint-planner/manage-azure-devops-work-items.mdsrc/pages/docs/test-management/atto/sprint-planner/manage-clickup-tasks.mdsrc/pages/docs/test-management/atto/sprint-planner/manage-linear-cycles.mdsrc/pages/docs/test-management/atto/sprint-planner/manage-sprints.mdsrc/pages/docs/test-management/imports-and-exports/csv-file.mdsrc/pages/docs/test-management/integrations/azure-two-way-integration/manage-test-cases-in-azure-devops.mdsrc/pages/docs/test-management/integrations/azure-two-way-integration/manage-test-runs-in-azure-devops.mdsrc/pages/docs/test-management/integrations/azure-two-way-integration/testsigma-integration-app-for-azuredevops.mdsrc/pages/docs/test-management/integrations/jira-two-way-integration/install-and-connect-testsigma.mdsrc/pages/docs/test-management/integrations/jira-two-way-integration/manage-test-cases-in-jira.mdsrc/pages/docs/test-management/integrations/jira-two-way-integration/manage-test-runs-in-jira.mdsrc/pages/docs/test-management/projects/manage-projects.mdsrc/pages/docs/test-management/settings/audit-logs.mdsrc/pages/docs/test-management/settings/notifications/email.mdsrc/pages/docs/test-management/settings/notifications/microsoft-teams.mdsrc/pages/docs/test-management/settings/notifications/overview.mdsrc/pages/docs/test-management/settings/notifications/slack.mdsrc/pages/docs/test-management/test-capture/capture-logs.mdsrc/pages/docs/test-management/test-capture/install-and-configure.mdsrc/pages/docs/test-management/testsigma-two-way-integration/connect-testsigma.mdsrc/pages/docs/test-management/testsigma-two-way-integration/execute-test-plans.md
Comment on lines
+180
to
+193
|  | ||
|
|
||
| Refining ensures each test case is actionable, traceable, and ready for manual execution or automation. | ||
|
|
||
| ### **2. Work Within a Single Feature Context** | ||
| To get accurate, focused, and manageable test case generation from Atto, always create a session in Atto’s Home around a single feature or a group of closely related features. | ||
| - Group relevant assets such as user stories, Figma designs, API endpoints, and files related to that one feature. | ||
|  | ||
|  | ||
| - Avoid mixing unrelated contexts in the same session to prevent confusing or inaccurate outputs. | ||
| - Use the prompt area effectively by clearly stating: | ||
| - The feature under test | ||
| - The context you’re providing (stories, designs, workflows, etc.) | ||
| - The expected outcomes from the AI-generated test cases | ||
|  | ||
|  |
There was a problem hiding this comment.
🧩 Analysis chain
🏁 Script executed:
#!/bin/bash
set -euo pipefail
# Collect all migrated URLs from docs
urls=$(rg -no 'https://s3\.amazonaws\.com/website-static-docs\.testsigma\.com/[^\s)\]"]+' src/pages/docs/test-management | awk '{print $2}' | sort -u)
echo "Checking migrated image URLs..."
fail=0
while IFS= read -r u; do
code=$(curl -s -o /dev/null -w "%{http_code}" -I "$u")
echo "$code $u"
if [ "$code" -lt 200 ] || [ "$code" -ge 400 ]; then
fail=1
fi
done <<< "$urls"
if [ "$fail" -ne 0 ]; then
echo "One or more URLs are not reachable (non-2xx/3xx)."
exit 1
fi
echo "All migrated URLs resolved successfully."Repository: testsigmahq/TMSDocs
Length of output: 94
🏁 Script executed:
# First, verify the file exists and view the relevant section
ls -la src/pages/docs/test-management/atto/best-practices/generating-test-cases.mdRepository: testsigmahq/TMSDocs
Length of output: 187
🏁 Script executed:
# Extract the specific URLs from the file
rg 'https://s3\.amazonaws\.com/website-static-docs\.testsigma\.com/[^\s)\]"]+' src/pages/docs/test-management/atto/best-practices/generating-test-cases.mdRepository: testsigmahq/TMSDocs
Length of output: 490
🏁 Script executed:
# Test HTTP status of the URLs found
#!/bin/bash
set -euo pipefail
# Extract unique URLs from the specific file
urls=$(rg 'https://s3\.amazonaws\.com/website-static-docs\.testsigma\.com/[^\s)\]"]+' src/pages/docs/test-management/atto/best-practices/generating-test-cases.md -o | sort -u)
if [ -z "$urls" ]; then
echo "No URLs found in the file."
exit 0
fi
echo "Testing migrated image URLs..."
fail=0
while IFS= read -r u; do
code=$(curl -s -o /dev/null -w "%{http_code}" -m 5 -I "$u")
echo "$code $u"
if [ "$code" -lt 200 ] || [ "$code" -ge 400 ]; then
fail=1
fi
done <<< "$urls"
if [ "$fail" -ne 0 ]; then
echo "One or more URLs are not reachable (non-2xx/3xx)."
exit 1
fi
echo "All migrated URLs are accessible."Repository: testsigmahq/TMSDocs
Length of output: 512
Migrated image URLs in this file are returning HTTP 403 (Forbidden), not HTTP 200.
All three images referenced in src/pages/docs/test-management/atto/best-practices/generating-test-cases.md (lines 180-193) return 403 status:
https://s3.amazonaws.com/website-static-docs.testsigma.com/new_images/projects/Atto_Testsigma/Refine_Tests_with_AI.pnghttps://s3.amazonaws.com/website-static-docs.testsigma.com/new_images/projects/Atto_Testsigma/All_Inputs_for_AI.pnghttps://s3.amazonaws.com/website-static-docs.testsigma.com/new_images/projects/Atto_Testsigma/Clear_Prompt_Example.png
These broken image links must be resolved before merge.
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@src/pages/docs/test-management/atto/best-practices/generating-test-cases.md`
around lines 180 - 193, The three image links in
src/pages/docs/test-management/atto/best-practices/generating-test-cases.md (the
markdown image references to Refine_Tests_with_AI.png, All_Inputs_for_AI.png,
and Clear_Prompt_Example.png) are returning HTTP 403; replace or fix them so
they serve with HTTP 200: verify the images exist in the S3 bucket or CDN and
correct their public path/ACL, or copy the images into the repo's static assets
and update the three markdown image URLs to the repo-relative paths; after
updating, verify each URL returns 200 and update the PR accordingly.
bharathk08
added a commit
that referenced
this pull request
Apr 20, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary by CodeRabbit