chore: removed eslint-disable statements

[sohailfatima]

Description

Removed all unnecessary / unintentional eslint-disable statements, refactoring the code accordingly. No changes have been made to the functionality.

[openedx-webhooks]

Thanks for the pull request, @sohailfatima! Please note that it may take us up to several weeks or months to complete a review and merge your PR.

Feel free to add as much of the following information to the ticket as you can:

• supporting documentation
• Open edX discussion forum threads
• timeline information ("this must be merged by XX date", and why that is)
• partner information ("this is a course on edx.org")
• any other information that can help Product understand the context for the PR

All technical communication about the code itself will be done via the GitHub pull request interface. As a reminder, our process documentation is here.

Please let us know once your PR is ready for our review and all tests are green.

⚠️ We can't start reviewing your pull request until you've submitted a signed contributor agreement or indicated your institutional affiliation. Please see the CONTRIBUTING file for more information. If you've signed an agreement in the past, you may need to re-sign. See The New Home of the Open edX Codebase for details.

Once you've signed the CLA, please allow 1 business day for it to be processed. After this time, you can re-run the CLA check by adding a comment here that you have signed it. If the problem persists, you can tag the @openedx/cla-problems team in a comment on your PR for further assistance.

[mphilbrick211]

Thank you for this contribution, @sohailfatima! Please let me know if you have any questions regarding submitting the CLA form. Thanks!

[codecov]

Codecov Report

Attention: 22 lines in your changes are missing coverage. Please review.

Comparison is base (88a985d) 92.39% compared to head (f4d7f3e) 92.76%.

Files	Patch %	Lines
src/discussions/posts/data/slices.js	78.12%	14 Missing ⚠️
src/discussions/post-comments/data/slices.js	95.65%	3 Missing ⚠️
...iscussions/post-comments/comments/CommentsView.jsx	85.71%	1 Missing and 1 partial ⚠️
src/discussions/data/slices.js	83.33%	1 Missing ⚠️
src/discussions/topics/data/slices.js	85.71%	1 Missing ⚠️
.../discussions/topics/topic-group/TopicGroupBase.jsx	0.00%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #646      +/-   ##
==========================================
+ Coverage   92.39%   92.76%   +0.37%     
==========================================
  Files         169      156      -13     
  Lines        3456     3303     -153     
  Branches      896      902       +6     
==========================================
- Hits         3193     3064     -129     
+ Misses        243      219      -24     
  Partials       20       20              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[awais-ansari]

1. Remove all the icon components and use these icons from paragon.
2. Remove eslint-disable no-param-reassig disable eslint and update the code.
3. Remove all the index files that are just using for exporting one component.

[awais-ansari]

Please test Discussions MFE properly once again because this change is big. Other than this PR LGTM.

[awais-ansari]

@sohailfatima Please rebase this branch.

[sundasnoreen12]

nit: few minor changes

[sundasnoreen12]

PR LGTM. Please rebase with master

[awais-ansari]

I'm closing this PR as @sohailfatima does not have the cla permission. We created a new PR for these Changes. 658

[openedx-webhooks]

@sohailfatima Even though your pull request wasn’t merged, please take a moment to answer a two question survey so we can improve your experience in the future.