Should strict mode be turned off on Server Side Rendering (SSR)?

[codeBelt]

Should I be worried by MobX warnings on server-side? I am using Next.js and I am thinking strict mode should only be in the browser?

const isServer = typeof window === 'undefined';
const isBrowser = typeof window !== 'undefined';

enableStaticRendering(isServer);

configure({
  enforceActions: 'always',
  computedRequiresReaction: isBrowser,
  reactionRequiresObservable: isBrowser,
  observableRequiresReaction: isBrowser,
  disableErrorBoundaries: isBrowser,
});

[danielkcz]

What MobX warnings you have in mind? Please be as specific as possible.

[codeBelt]

Sorry, below is a sample of warnings.

Also here is the branch if you wanted to check out https://github.com/codeBelt/mobx-local-global-stores/tree/mobx-stores

[mobx] Observable [email protected]? being read outside a reactive context
[mobx] Observable [email protected]? being read outside a reactive context
[mobx] Observable [email protected]? being read outside a reactive context
[mobx] Observable [email protected]? being read outside a reactive context
[mobx] Observable [email protected]? being read outside a reactive context
[mobx] Observable [email protected]? being read outside a reactive context
[mobx] Observable [email protected]? being read outside a reactive context
[mobx] Computed value [email protected] is being read outside a reactive context. Doing a full recompute
[mobx] Observable [email protected] being read outside a reactive context
[mobx] Observable [email protected] being read outside a reactive context
[mobx] Observable [email protected]? being read outside a reactive context
[mobx] Observable [email protected]? being read outside a reactive context
[mobx] Observable [email protected]? being read outside a reactive context
[mobx] Observable [email protected]? being read outside a reactive context
[mobx] Computed value [email protected] is being read outside a reactive context. Doing a full recompute
[mobx] Observable [email protected] being read outside a reactive context
[mobx] Observable [email protected] being read outside a reactive context
[mobx] Observable [email protected] being read outside a reactive context
[mobx] Observable [email protected] being read outside a reactive context
[mobx] Observable [email protected]? being read outside a reactive context
[mobx] Computed value [email protected] is being read outside a reactive context. Doing a full recompute
[mobx] Observable [email protected] being read outside a reactive context
[mobx] Observable [email protected] being read outside a reactive context

[mweststrate]

Yeah I'd disable it on SSR, not having a reactive context is pretty much what you want server side :)

…

On Tue, Jan 19, 2021 at 3:43 PM Robert S. ***@***.***> wrote: Sorry, below is a sample of warnings. Also here is the branch if you wanted to check out https://github.com/codeBelt/mobx-local-global-stores/tree/mobx-stores [mobx] Observable ***@***.***? being read outside a reactive context [mobx] Observable ***@***.***? being read outside a reactive context [mobx] Observable ***@***.***? being read outside a reactive context [mobx] Observable ***@***.***? being read outside a reactive context [mobx] Observable ***@***.***? being read outside a reactive context [mobx] Observable ***@***.***? being read outside a reactive context [mobx] Observable ***@***.***? being read outside a reactive context [mobx] Computed value ***@***.*** is being read outside a reactive context. Doing a full recompute [mobx] Observable ***@***.*** being read outside a reactive context [mobx] Observable ***@***.*** being read outside a reactive context [mobx] Observable ***@***.***? being read outside a reactive context [mobx] Observable ***@***.***? being read outside a reactive context [mobx] Observable ***@***.***? being read outside a reactive context [mobx] Observable ***@***.***? being read outside a reactive context [mobx] Computed value ***@***.*** is being read outside a reactive context. Doing a full recompute [mobx] Observable ***@***.*** being read outside a reactive context [mobx] Observable ***@***.*** being read outside a reactive context [mobx] Observable ***@***.*** being read outside a reactive context [mobx] Observable ***@***.*** being read outside a reactive context [mobx] Observable ***@***.***? being read outside a reactive context [mobx] Computed value ***@***.*** is being read outside a reactive context. Doing a full recompute [mobx] Observable ***@***.*** being read outside a reactive context [mobx] Observable ***@***.*** being read outside a reactive context — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#2731 (reply in thread)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAN4NBEC5AX6GZRSH44U7ETS2WSDZANCNFSM4WI3Z66Q> .

[danielkcz]

But he has it disabled apparently unless the window happens to be defined on the server too :)

[mweststrate]

I guess, given the warnings :)

[codeBelt]

Sorry for the confusion. I was getting the errors on the server when doing:

const isServer = typeof window === 'undefined';

enableStaticRendering(isServer);

configure({
  enforceActions: 'always',
  computedRequiresReaction: true,
  reactionRequiresObservable: true,
  observableRequiresReaction: true,
  disableErrorBoundaries: true,
});

But I changed it to use isBrowser (like in first code example) after I saw mweststrate comment:

Yeah I'd disable it on SSR, not having a reactive context is pretty much what you want server side :)

Going forward I will be using the example in my original question.

Thanks for the help!