sttm-desktop
sttm-desktop copied to clipboard
Code Clean up and minor refactor
Code cleanup tasks
- [ ] Should add proptypes to the files where it’s missing
- [ ] File: SearchContent.jsx line:62 we are using static string ‘all’ instead that should some from banidb.js file. Should replace the static strings at all the places.
- [x] File: MiscHeader.jsx line: 51 we are using static string this should some from i18n. Issues like that should be fixed, not limited to this file only.
- [ ] File: MiscHeader.jsc line line: 36 we are passign static string to setTab file. Instead there should be a common js file in constants folder which should have all these tabs name and replace that variable in all the files that are using this static string. Not limited to the mentioned line number and file.
- [x] File: shortcut-functions.js is empty. Should be removed.
- [x] File: shortcut-map.json is no longer in use. Either we should find a way to use that again by separating the keyboard shortcut code from Launchpad.jsx file or remove it
- [ ] File: shortcut_tray.json is not in use. Find a way to get that in use or remove it
- [ ] File: viewer.json is empty. Remove or get in use
- [ ] File: .prettierrc maybe not in use. Get that in use again
- [ ] We can’t see new changes, without shutting down the server and starting again, it shouldn’t be this way. Some problem with how source maps are integrated; hot reload
- [ ] Rename the content1, content2, content3 to something more specific https://github.com/KhalisFoundation/sttm-desktop/pull/1969#discussion_r1634584100