mistercrunch opened a new pull request, #34850:
URL: https://github.com/apache/superset/pull/34850

   ## Summary
   
   Fixes critical dashboard import failures and implements complete theme 
import/export functionality. Dashboard imports were completely broken when 
themes were referenced - this PR resolves that and adds full theme portability 
across Superset instances.
   
   ## What Changed
   
   ### 🔧 Dashboard Import/Export Fixes
   - **Fixed dashboard import failures** when themes are referenced using 
proper UUID→ID mapping
   - **Auto-bundle themes** with dashboard exports for complete portability
   - **Export `theme_uuid` instead of `theme_id`** for cross-system 
compatibility
   - **Graceful fallback** when themes are missing during import
   
   ### 🎯 Enhanced Error Reporting  
   - **Detailed validation messages** showing exactly which file and validation 
failed
   - **Enhanced theme validation** with better debugging context
   - **Improved import error messages** with specific file names and error 
details
   
   ## Impact
   
   **Before**: Dashboard imports failed completely when themes were referenced  
   **After**: Dashboard imports succeed with proper theme linking and complete 
portability
   
   ## Testing
   
   - ✅ **13/13 theme integration tests passing**
   - ✅ **Dashboard export with theme bundling**: Working
   - ✅ **Dashboard import with theme UUID resolution**: Fixed and working  
   - ✅ **Dashboard import missing theme graceful fallback**: Working
   
   ## Technical Details
   
   ### Key Files
   - `superset/commands/dashboard/export.py` - Auto-bundle themes with exports
   - `superset/commands/dashboard/importers/v1/__init__.py` - Theme UUID→ID 
mapping logic
   - `superset/models/dashboard.py` - Added theme_id to import fields
   - `superset/dashboards/schemas.py` - Added theme_id to import schema
   
   ### Export Structure Example
   ```
   dashboard_export/
   ├── dashboards/My_Dashboard.yaml    # includes theme_uuid
   ├── themes/My_Theme.yaml            # AUTO-BUNDLED
   └── ...
   ```
   
   Resolves dashboard import failures and implements complete theme 
import/export system.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to