-
-
Notifications
You must be signed in to change notification settings - Fork 23
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
Installation error on windows #45
Comments
Please make sure you have the latest version installed |
I had the latest version but apparantly this doesnt work with the microsoft store version of slack, probably because it is installed in a different directory |
I don't have the microsoft version of slack and I get the same error, I heard in similar issues it might be because of Electron-DB? |
I was able to fix this locally on a Windows 10 install. The problem I saw is that this is due to the latest version number of slack and how the code resolves the slack directory. On Windows 10, the directory the code wants to find is function getWindowsDirectory() {
for (let i = 9; i >= 0; --i) {
for (let j = 9; j >= 0; --j) {
const ver = `4.${i}.${j}`; Locally I changed this code to the following and was able to run function getWindowsDirectory() {
for (let i = 11; i >= 0; --i) {
for (let j = 11; j >= 0; --j) {
const ver = `4.${i}.${j}`; Thanks for the work on this project, @mallowigi. I stare at slack so much throughout the day and am grateful for this awesome theme solution! |
Makes sense. I'd appreciate if you could make a pr. |
Just for completing @jlevier's solution, the |
I'm getting this error when running on windows:
The text was updated successfully, but these errors were encountered: