Doc fix: observable fields is not created

[PredokMiF]

Observable fields is not created if it placed after makeAutoObservable

[changeset-bot]

⚠️ No Changeset found

Latest commit: fb3f57d

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

[mweststrate]

If you run into this, your compilation setup is incorrect, the declarations at lines 32-34 should have already instantiated the fields in spec compliant class compilation. Please see the setup docs for the right transpiler options.

[PredokMiF]

Ok, I will create sandbok for the case to try code

[PredokMiF]

Yes, you are right. Some weird default TS settings for vite template react-ts. But from the learning point of view, my MR is still fair, because after makeAutoObservable in strict mode you can only change the referenced values in actions. So it is easier to assign first and then call makeAutoObservable to prevent warnings in console.

[mweststrate]

That works for simple constructors, but it introduces its own set of problems. E.g. if you introduce branching in the constructor, or call local methods, they might now obtain references to objects before they are turned into observables, etc.

[PredokMiF]

Understood. Thank you for your prompt response and clarification. I will close the PR

[PredokMiF]

It is realy bug of vite. The workaround is contained in the issue