Feature/bilingual i18n with tests#1
Open
picspin wants to merge 16 commits into
Open
Conversation
added 7 commits
August 19, 2025 08:33
…suite - Add i18n system with Chinese as default and English toggle - Implement LanguageToggle component with Globe icon - Update App, Header, and InputSection components to use translations - Add comprehensive test suite with 49 passing tests: * Business logic tests (calculations, ROI, formatting) * Data management tests (device specs, validation) * State management tests (Zustand store) * Component tests (App navigation, UI) - Add Vitest testing framework with React Testing Library - Include test coverage reporting with V8 - Add testing documentation and guides - 100% coverage on core business logic - All tests passing with zero failures Features: - Bilingual UI (中文/English) with language toggle - Comprehensive ROI calculations for medical equipment - Device comparison and analysis - Professional test suite for enterprise deployment
- Document bilingual support (Chinese/English) with language toggle - Explain comprehensive test suite (49 tests, 100% core coverage) - Provide quick start guide and usage instructions - Detail technical implementation and architecture - Include quality metrics and deployment readiness - Ready for production use in healthcare environments
- Fix App tests to include I18nProvider wrapper - Use role-based queries to avoid text conflicts - Add within() helper for specific element scoping - All 49 tests now passing with bilingual functionality - Tests validate both Chinese (default) and navigation behavior
…etercomparison components
- All text elements use the translation system for bilingual support - Tooltips in the radar chart properly display translated content - The analysis conclusion section uses translated content with dynamic values - All charts and components maintain their bilingual functionality
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
✅ Deploy Preview for centargo-roi ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
added 2 commits
September 3, 2025 11:53
- Added percentage input field (0-100%) in InputSection - Includes helper text explaining the purpose Validates input range with visual percentage indicator - Updated Calculation Logic: Modified calculateDeltaP() to factor in CT enhancement rate - Only enhanced scans benefit from time savings (multiplied by enhancement rate) - Updated all calculation functions to use the new parameter Additional Revenue Calculation: - New calculateAdditionalRevenue() function - Calculates potential revenue from saved time distributed by enhancement rate - Uses national CT fee standards - Accounts for different exam durations New Metric Card: - Purple-themed "Potential Additional Revenue" card in ResultsSection - Shows monthly potential additional revenue from saved time - Responsive grid layout (4 columns on large screens)
Regarding some platform, add a verification.txt file is necessary.
added 2 commits
September 17, 2025 00:48
…m/picspin/ROICalc into feature/bilingual-i18n-with-tests
Add echarts and echarts-for-react dependencies to enable interactive charting capabilities for displaying radiology equipment ROI comparisons. The ECharts library will provide advanced visualization features for comparing financial metrics between baseline and target equipment configurations.
- Import and integrate CumulativeRevenueChart component into results layout - Refactor chart grid to include full-width cumulative revenue chart - Fix monthly time saved calculation using consistent methodology - Add comprehensive integration tests for AdditionalRevenueCard translations - Implement language switching tests for I18n provider functionality
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
1 participant
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.
Add more adverse highlight such as contrast volume to make this tool more comprehensive, and build bilingual mode for i18n usres.