fix: correct spelling of "TradeTrust" in various components#1128
fix: correct spelling of "TradeTrust" in various components#1128manishdex25 wants to merge 2 commits intomasterfrom
Conversation
✅ Deploy Preview for reference-implementation ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
📝 WalkthroughWalkthroughThis PR updates branding text from "Tradetrust" to "TradeTrust" (proper capitalization) across component labels, documentation, and test selectors. No functional logic, routing, or component structure changes are included. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (2 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 |
![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
🤖 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/components/CertificateDropZone/LoadDemoCertificate.tsx`:
- Line 20: Replace the remaining incorrect capitalization occurrences of the
document name "Tradetrust-bill-of-lading.tt" with the correct
"TradeTrust-bill-of-lading.tt" across the FormTransferableRecordPanel code and
its tests/stories; specifically update the string literal inside the
FormTransferableRecordPanel component (component name:
FormTransferableRecordPanel), the corresponding test assertions in
FormTransferableRecordPanel.test (update any expect/match/text selectors), and
the example in FormTransferableRecordPanel.stories (update displayed/prop values
and any snapshot text) so all references use "TradeTrust-bill-of-lading.tt"
consistently.
🪄 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: 589be21b-2c62-4e4e-bc3a-3536a1bc7645
📒 Files selected for processing (3)
src/components/CertificateDropZone/LoadDemoCertificate.tsxsrc/components/NavigationBar/NavigationBar.stories.tsxsrc/test/demo-sample.spec.ts
| }} | ||
| > | ||
| View Demo Tradetrust Document | ||
| View Demo TradeTrust Document |
There was a problem hiding this comment.
🧩 Analysis chain
🏁 Script executed:
#!/bin/bash
# Search for remaining instances of "Tradetrust" (incorrect capitalization)
# Exclude common directories and look for the old branding
rg -i 'tradetrust' \
--type ts --type tsx --type js --type jsx \
-g '!node_modules' -g '!dist' -g '!build' \
| rg -v 'TradeTrust' \
| rg 'Tradetrust'Repository: TradeTrust/tradetrust-website
Length of output: 100
🏁 Script executed:
#!/bin/bash
# Check for remaining instances of "Tradetrust" (incorrect capitalization)
# using correct rg syntax
rg 'Tradetrust' --type ts -g '!node_modules' -g '!dist' -g '!build'Repository: TradeTrust/tradetrust-website
Length of output: 789
🏁 Script executed:
# View the actual file content at the lines mentioned
cat -n src/components/CertificateDropZone/LoadDemoCertificate.tsx | sed -n '15,30p'Repository: TradeTrust/tradetrust-website
Length of output: 669
🏁 Script executed:
# Also check the test file mentioned in the comment
cat -n src/test/demo-sample.spec.ts | sed -n '25,35p'Repository: TradeTrust/tradetrust-website
Length of output: 470
LGTM! Branding text updated correctly.
The button labels in LoadDemoCertificate.tsx have been updated to use the correct "TradeTrust" capitalization, and the corresponding test selector has been updated to match.
However, verification found remaining instances of incorrect capitalization ("Tradetrust-bill-of-lading.tt") in the FormTransferableRecordPanel component files that should also be updated to "TradeTrust-bill-of-lading.tt" for consistency:
src/components/Creator/FormTransferableRecordPanel/FormTransferableRecordPanel.tsxsrc/components/Creator/FormTransferableRecordPanel/FormTransferableRecordPanel.test.tsxsrc/components/Creator/FormTransferableRecordPanel/FormTransferableRecordPanel.stories.tsx
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@src/components/CertificateDropZone/LoadDemoCertificate.tsx` at line 20,
Replace the remaining incorrect capitalization occurrences of the document name
"Tradetrust-bill-of-lading.tt" with the correct "TradeTrust-bill-of-lading.tt"
across the FormTransferableRecordPanel code and its tests/stories; specifically
update the string literal inside the FormTransferableRecordPanel component
(component name: FormTransferableRecordPanel), the corresponding test assertions
in FormTransferableRecordPanel.test (update any expect/match/text selectors),
and the example in FormTransferableRecordPanel.stories (update displayed/prop
values and any snapshot text) so all references use
"TradeTrust-bill-of-lading.tt" consistently.
There was a problem hiding this comment.
No need for file name changes
There was a problem hiding this comment.
@manishdex25, understood! The "Tradetrust-bill-of-lading.tt" references are actual file names/identifiers, so leaving them unchanged is the right call — renaming them could break functionality. The scope of this PR is correctly limited to fixing visible UI display text only.
✏️ Learnings added
Learnt from: manishdex25
Repo: TradeTrust/tradetrust-website PR: 1128
File: src/components/CertificateDropZone/LoadDemoCertificate.tsx:20-20
Timestamp: 2026-04-09T07:28:10.229Z
Learning: In the TradeTrust/tradetrust-website repository, the filename literal "Tradetrust-bill-of-lading.tt" (used in FormTransferableRecordPanel component and its test/stories files) should NOT be renamed to "TradeTrust-bill-of-lading.tt". It is an actual file name/identifier, not display text, and must remain unchanged. Only visible UI text labels need the correct "TradeTrust" capitalization.
Note: Learnings are effective only in the context of similar code segments. To apply general coding guidelines, please configure review instructions. You can manage existing learnings in the app.
❌ Deploy Preview for tradetrust-dev failed. Why did it fail? →
|
1 participant
Summary
What is the background of this pull request?
Changes
Issues
What are the related issues or stories?
Summary by CodeRabbit
Style
Tests