Remove index files - Githubissues

wso2 / identity-apps

Apache License 2.0

69 stars 259 forks source link

Remove index files #7048

Closed pavinduLakshan closed 1 day ago

pavinduLakshan commented 2 weeks ago

Purpose

This PR removes some index files used in several feature modules and updates the imports in the dependent packages to import directly from the file where entities are defined.

Why barrel files (such as index.js) should not be used: https://medium.com/@alonmiz1234/why-you-should-avoid-index-js-3321a9902120

Related Issues

N/A

Related PRs

N/A

Checklist

[ ] e2e cypress tests locally verified. (for internal contributers)
[x] Manual test round performed and verified.
[ ] UX/UI review done on the final implementation.
[ ] Documentation provided. (Add links if there are any)
[ ] Relevant backend changes deployed and verified
[ ] Unit tests provided. (Add links if there are any)
[ ] Integration tests provided. (Add links if there are any)

Security checks

[x] Followed secure coding standards in http://wso2.com/technical-reports/wso2-secure-engineering-guidelines
[ ] Ran FindSecurityBugs plugin and verified report
[x] Confirmed that this PR doesn't commit any keys, passwords, tokens, usernames, or other secrets

wso2-jenkins-bot commented 3 days ago

⚠️ No Changeset found

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go.

If these changes should result in a version bump, you need to add a changeset.

Refer Release Documentation to learn how to add a changeset.