Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: refactor Secret Manager Overview page #3009

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

moizsheraz
Copy link

@moizsheraz moizsheraz commented Jan 19, 2025

Create a new directory for PageOverview component
Update all imports to reflect the new structure

Description 📣

This PR refactors the structure of the Secret Manager Overview page by moving the ProductOverviewPage component to its own directory. The component was previously being used in a very large component, which made the code harder to maintain and follow. It was more logical and cleaner to place it in a dedicated file. Hence, a new directory was created for this component to better organize the structure. All imports have been updated to reflect the new directory structure, ensuring that the application works seamlessly after the refactor.

Type ✨

  • Bug fix
  • New feature
  • Improvement
  • Breaking change
  • Documentation

Tests 🛠️

The changes have been tested by running the application and confirming that the Secret Manager Overview page loads correctly. The component is now correctly imported and used in the updated file structure.

# To test, simply run the application locally and navigate to the Secret Manager Overview page.
npm run dev

- Create a new directory for PageOverview component
- Update all imports to reflect the new structure
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant