appsmithorg / appsmith

Platform to build admin panels, internal tools, and dashboards. Integrates with 25+ databases and any API.
https://www.appsmith.com
Apache License 2.0
33.79k stars 3.63k forks source link

(JS-0458) Definitions for unused states detected #7943

Open rishabhrathod01 opened 2 years ago

rishabhrathod01 commented 2 years ago

Description

You should avoid defining states that are not being used because unused states might affect while refactoring the codebase and introduces unnecessary operations by ReactJS. It is recommended to plan the behavior and operation of the component.

Occurrences

There are 9 occurrences of this issue in the repository.

See all occurrences on DeepSource → deepsource.io/gh/appsmithorg/appsmith/issue/JS-0458/occurrences/

Hard-Coder05 commented 2 years ago

@Rishabh-Rathod I would like to work on this issue. Please assign me this!

evergreenx commented 2 years ago

@Rishabh-Rathod i will like to work on this

mohanarpit commented 2 years ago

@evergreenx Thank you for showing interest in Appsmith.

Sorry, this issue has already been claimed. I would request you to please look at some of our other Good First Issues

ampatzidhs commented 1 year ago

I am interested for this issue, would you assign me ?

rishabhrathod01 commented 1 year ago

@ampatzidhs thanks for showing interest 🎉 , This is all yours. Assigning this to you now.

Please don't forget to read the Contribution Guidelines. Would appreciate it if you can open a PR within the next 4 days.

priyanka-mahour commented 1 year ago

@Rishabh-Rathod can you please assign this to me?

rishabhrathod01 commented 1 year ago

@priyanka-mahour Greetings, thanks for showing interest 🎉 , This is all yours. Assigning this to you now.

Please don't forget to read the Contribution Guidelines. Would appreciate it if you can open a PR within the next few days.

riteshsp2000 commented 1 year ago

Hello @Rishabh-Rathod

I noticed that this issue is still open and the previous PR that was opened for the issue was closed. The unused states still persist. I'd like to raise a PR fixing this, could you please assign me the issue?

harjotjuneja6 commented 9 months ago

Hi @rishabhrathod01 , can I work on this issue.

PrateekKrishna commented 8 months ago

Hey @rishabhrathod01 is this issue is still open, i'd like to work on this