Grammarly is not changing the dialect about grammarly HOT 17 OPEN

I just downgraded from 0.24.0 to 0.22.1 and this issue disappeared. There is a bug in the 0.24.0 version

from grammarly.

Hope this gets fixed soon.

from grammarly.

Same here. I Downgraded to v.0.23.10, and it's all good now. Hope this will fixed in the next release

from grammarly.

Still happens here too

from grammarly.

I am encountering this issue, too, and I would love to see it fixed. Thank you!

from grammarly.

That, I don't know. The Grammarly account is probably owned by @znck. So he may have insights about it.

EDIT:
It's honestly surprising that it still works...

from grammarly.

I doesn't change from American even when I set it to Australian, Canadian, ...

from grammarly.

After downgrading, the extension started to report "failed to connect to language server" and stopped working. I've had to re-install latest version and the dialect changes are being ignored again

from grammarly.

Is there any fix to this?

from grammarly.

I am pretty stupid when it comes to JavaScript, let alone VSCode extensions. So I am not able to fix it 😕

from grammarly.

Also coming across this, would love it for be fixed!

from grammarly.

Worryingly, I switched to the pre-release version v0.25.0 and the issue is still present. Hopefully, this can be fixed before the proper release of the next version.

from grammarly.

While looking into this, I realized that Grammarly has shut down their developer SDK as explained on the website: "Grammarly for Developers and the Text Editor SDK were discontinued on January 10, 2024" (https://developer.grammarly.com/).

So, this extension may work correctly and Grammarly may not. In any case, this is a real bummer...

@znck, Do you have more information or ideas for the future or this extension?

from grammarly.

Maybe it has to do with this?

from grammarly.

Maybe it has to do with this?

grammarly/packages/grammarly-languageserver/src/services/ConfigurationService.ts

Line 36 in a81aac2

documentDialect: 'american',

This seems to be the default. If 'result?.config' contains the dialect it will "overwrite" the default. If, for some reason, the dialect is not in the 'result?.config' it will of course always use the American dialect. Just note my previous comment. It might not be the code at fault.

from grammarly.

Interesting. But how come the American spelling is the only one still supported though, whilst ignoring the documentDialect property?

from grammarly.

After git diffing versions 0.24.0 and 0.22.1, it looks like the main changes that could potentially be related to this issue are on the ConfigurationService.ts file and the Grammarly SDK version in packages/grammarly-languageclient/package.json

In ConfigurationService.ts, it seems to me that all the properties from config except documentDialect are being included in DocumentConfig, which might be the reason why the chosen dialect is being ignored. However I'm not very experienced in TS syntax so I'm not sure if this is case.

const options: DocumentConfig = {
  documentDialect: 'american',
  ...result?.config,
}

This might fix it.

const options: DocumentConfig = {
  ...result?.config,
}

// Use the user's settings if it contains documentDialect and otherwise default to "american".
options.documentDialect = options.documentDialect ?? 'american';

from grammarly.