Merging Develop into Master - Dec 2nd, 2025#2799
Merged
Merged
Conversation
Changed 'model' resource type to query 'model_inventories' table instead of 'model_files'. This was causing 404 errors when accessing shared model views even though models existed in the database.
Resolved merge conflicts between GroupBy and Export features by keeping both: Files resolved: - Tasks/index.tsx: Added both GroupBy and Export imports/logic - Vendors/index.tsx: Added both GroupBy and Export imports/logic - TrainingRegistar/index.tsx: Added both GroupBy and Export imports/logic - ModelInventory/index.tsx: Added both GroupBy and Export imports/logic (3 conflicts) - IncidentManagement/index.tsx: Added both GroupBy and Export imports - ProjectsList/ProjectsList.tsx: Added both GroupBy and Export imports Both features are now functional and complementary. GroupBy allows users to organize table data by various fields, while Export allows users to download table data in CSV/PDF/Excel formats.
- Changed header from h5 to h6 for smaller size - Filter table columns to show only Model Inventory fields: provider, model, version, approver, security_assessment, status, status_date - Add formatCellValue function for proper field formatting: - security_assessment: Yes/No badge with green/red colors - status: Colored badge based on status value - status_date: Formatted as readable date - Convert "Powered by VerifyWise" to clickable link (verifywise.ai) - Add uppercase styling to table headers to match Model Inventory
Changed shareAllFields default from false to true in ModelInventory page. This ensures that when sharing the model inventory table view, all fields (provider, model, version, approver, security_assessment, status, status_date) are included in the shared data instead of being filtered to just basic fields (id, name, description). Users will need to create a new share link or refresh their existing link for this change to take effect.
- Remove unused Stack import - Remove unused paginatedData variable - Fix property name: model.model_name -> model.name (MLFlowModel uses 'name', not 'model_name')
- Add shareLinkId state to track the current share link ID - Store share link ID when creating a new share link - Update handleShareSettingsChange to use updateShareMutation when shareLinkId exists, allowing settings changes to update the existing link instead of requiring a manual refresh - Show success/error alerts when settings are updated - Refresh button still creates a brand new link with new token All Settings now work correctly: - Share all fields: Backend filters data based on this setting - Allow data to be exported: Shows/hides export button in SharedView - Allow viewers to open records: Shows/hides open record button - Display toolbar: Shows/hides toolbar in SharedView
Changes: - Reorganized hamburger menu popup from single column to 2-column layout - Column 1 (Account): My profile with role, Restart onboarding, Logout - Column 2 (Explore VerifyWise): Help center, What's new, User community, Get support, Give feedback - Added light gray background (#F9FAFB) to Account column - Increased popup width from 220px to 500px for 2-column layout - Added new icons: User, HelpCircle, Newspaper, Users, Headphones - Improved hover states with appropriate background colors - Profile item now shows user role as subtitle - Logout button positioned at bottom of Account column with mt:auto
Changes: - Increased popup width from 500px to 560px (collapsed: 520px) - Added 16px padding around entire content (p: 2) - Increased column padding from 2.5 to 3 (24px) - Increased minimum height from 280px to 320px - Increased spacing between menu items from 0.5 to 1 (8px) - Increased button padding: px from 2 to 2.5, py from 1.5 to 2 - Increased gap between icon and text from 2 to 2.5 (20px) - Increased all icon sizes from 16px to 18px for better visibility - Added border radius to Account column background - Better visual hierarchy and breathing room
- Renamed User icon import to UserIcon to avoid conflict with User type - Updated usage in My profile button - Fixes 'Identifier User has already been declared' syntax error
- Added flex: 1 to Explore VerifyWise Stack to match Account column - Both columns now have identical spacing configuration - Ensures visual consistency between menu item gaps
Changes: - Reduced outer padding from 16px to exactly 8px (p: 1) - Changed button height to 32px to match sidebar items - Updated icon size from 18px to 16px to match sidebar - Changed gap from 20px to 16px (theme.spacing(4)) - Updated horizontal padding to 16px (theme.spacing(4)) - Set icon colors to theme.palette.text.tertiary (instead of inheriting) - Maintains hover state with green color (#13715B) - All menu items now have consistent styling with sidebar navigation
Replaced the standard button group in team settings with the same ButtonToggle pattern used in framework controls. This provides: - Consistent UI/UX across the application - Sliding background indicator animation - Improved visual feedback for filter selection - Unified styling with framework controls section
Replaced inline styled MUI Box components with the proper VerifyWise ButtonToggle component for team member role filters. This ensures: - Consistent use of VerifyWise components across the application - Cleaner, more maintainable code - Proper component abstraction and reusability - Updated filter state to use string type to match ButtonToggle API
1. Fix empty table handling:
- Changed to return empty array instead of 404 for table views
- Added 'No records to display' message in SharedView
- Log as 'info' instead of 'error' when table is empty
2. Fix field filtering for model_inventories:
- Added resource-type-specific field filtering
- Model inventories now show: provider, model, version, status
when shareAllFields=false (instead of trying to show non-existent
'name' and 'description' fields)
- Generic fallback maintained for other resource types
- SharedView dynamically detects available columns based on data
3. Improved SharedView column detection:
- Automatically filters out 'id' column from display
- Detects whether shareAllFields is enabled by checking for
approver field presence
- Shows appropriate columns based on available data
…e-team-filters-01GHpJqLm48RJfdgbqNFKCh7 Apply ButtonToggle component to team member filters
…onality-01GHpJqLm48RJfdgbqNFKCh7 Add comprehensive GroupBy functionality to data tables
- Changed from 2-column to single-column layout - Added two category sections: Account and Explore VerifyWise - Account section: My profile (with role), Restart onboarding - Explore VerifyWise: Help center, What's new?, User community, Get support, Give feedback - Added horizontal divider line before Logout - Adjusted drawer width for single column (260px/300px) - All menu items maintain 32px height with consistent styling - Categories are non-clickable section headers
Changed updateShareLink to use process.env.FRONTEND_URL (with fallback to localhost:5173) instead of hardcoded 'https://app.verifywise.com'. This ensures consistency with createShareLink and allows proper URL configuration across different environments.
logStructured only accepts 'error', 'processing', or 'successful' as the first parameter. Changed 'info' to 'successful' for empty table view logging.
- Removed "Restart onboarding" option from Account section - Added conditional "Create demo data" / "Delete demo data" option - Shows "Create demo data" when hasDemoData is false - Shows "Delete demo data" when hasDemoData is true - Uses Database icon for create, Trash2 icon for delete - Maintains consistent 32px height and styling
Removed the conditional hiding of the 'Send a view only link...' description text. Now it remains visible both when sharing is enabled and disabled, providing consistent context to users.
- Added user avatar next to name in expanded sidebar button area - Updated popup menu profile section to show: - User full name on left, role on right (same line) - Email underneath name - Adjusted profile section height to 48px minimum to accommodate two lines - Removed user icon from profile section as name/role/email are now shown
- Removed unused Grid import (switched to single column layout) - Removed unused UserIcon import (profile section no longer uses icon) - Removed unused RotateCcw import (Restart onboarding removed) - Removed unused resetOnboarding hook (Restart onboarding removed)
…-single-column-01GHpJqLm48RJfdgbqNFKCh7 Redesign user menu with single-column layout and profile improvements
- Removed the setting from ShareViewDropdown UI - This setting only works for single record views, not table views - For Model Inventory (table view), this was non-functional - Improved 'Display toolbar' help text to be more descriptive - Backend still supports the setting for backward compatibility
Add kubernetes deployment
…ion-settings Update organization settings
Comment out proxy settings for Vite dev server.
Contributor
|
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 You can disable this status message by setting the ✨ Finishing touches🧪 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 |
Fixing Polynomial regular expression used on uncontrolled data
…into alert-autofix-24
Potential fix for code scanning alert no. 24: Missing CSRF middleware
- Make form columns flexible (flex: 1) to fill available modal width - Change fixed 350px field widths to 100% within their columns - Fix Start date and Geography row to equally distribute and match Team members width - Update teamMembersSxStyle to use 100% width instead of fixed 350px - Ensure Target industry and Description fields align with columns above
Fix use case modal form field alignment
Enhancements on Risk Management and Task Management
HarshP4585
approved these changes
Dec 3, 2025
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
8 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.
Merging Develop into Master - Dec 2nd, 2025
Merging Develop into Master - Dec 2nd, 2025