90 - Remove old CSS templates views and endpoints

[dpgaspar]

Remove the already deprecated code: #28387

PR: #31942

[dosubot]

To remove old CSS templates, views, and endpoints as outlined in the provided code, follow these steps:

1. Identify Dependencies: Check both frontend and backend code for references to the CSS templates, views, or endpoints you plan to remove. This includes reviewing components and API endpoints that interact with CSS templates.

2. Update or Remove References: If any functionalities or components depend on the elements you're removing, update them to use alternatives or remove those functionalities if they're no longer needed.

3. Use the Bulk Delete Endpoint: For bulk deletion, use the endpoint /api/v1/css_template/?q= with a query parameter that specifies the IDs of the templates to be deleted. Ensure proper error handling and response messages during the deletion process.

4. Test After Removal: Thoroughly test your application to ensure no other functionalities are affected by the removal. This includes testing list view, creation, editing, and deletion functionalities related to CSS templates.

5. Commit Changes with Documentation: Once confirmed that the removal does not adversely affect other functionalities, commit your changes with clear documentation on what was removed and why.

By carefully following these steps, you can ensure a smooth removal process without negatively impacting your project's functionality.

Details

Sources

• superset/css_templates/api.py
• superset-frontend/src/pages/CssTemplateList/index.tsx
• superset/css_templates/schemas.py
• superset/views/css_templates.py
• superset/examples/css_templates.py
• superset/views/core.py
• superset/cli/test_db.py
• .github/workflows/superset-cli.yml
• .github/workflows/superset-e2e.yml
• tests/integration_tests/datasets/api_tests.py

About Dosu
This response is meant to be useful and save you time. It is not meant to be a precise solution, but rather a starting point for your own research.

Help me be more useful! Please leave a 👍 if this is helpful and 👎 if it is irrelevant.

To continue the conversation, mention @dosu.