You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
What is the new or updated feature that you are suggesting?
When you use createFileSessionStorage show a warning or throw if the dir is inside the app folder.
Why should this feature be included?
Right now you can put the file inside app, and if you do that and commit the session in a loader (e.g. to read a flas message) the commit will update the file and in development it will cause a reload, which will run the loader again and commit the session again causing the file to change again repeating the cycle and causing an infinite loop.
In production is not an issue, but the function should probably warn if you put the dir to be inside app to avoid this issue.
This discussion was converted from issue #1142 on April 19, 2022 22:29.
Heading
Bold
Italic
Quote
Code
Link
Numbered list
Unordered list
Task list
Attach files
Mention
Reference
Menu
reacted with thumbs up emoji reacted with thumbs down emoji reacted with laugh emoji reacted with hooray emoji reacted with confused emoji reacted with heart emoji reacted with rocket emoji reacted with eyes emoji
-
What is the new or updated feature that you are suggesting?
When you use createFileSessionStorage show a warning or throw if the dir is inside the app folder.
Why should this feature be included?
Right now you can put the file inside app, and if you do that and commit the session in a loader (e.g. to read a flas message) the commit will update the file and in development it will cause a reload, which will run the loader again and commit the session again causing the file to change again repeating the cycle and causing an infinite loop.
In production is not an issue, but the function should probably warn if you put the dir to be inside app to avoid this issue.
Beta Was this translation helpful? Give feedback.
All reactions