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
AstroPi - hide save popup/toast #965
Conversation
|
|
|
|
|
@@ -86,7 +86,7 @@ describe("When no user is in state", () => { | |||
project: { | |||
components: [], | |||
}, | |||
hasShownSavePrompt: false, | |||
hasShownSavePrompt: true, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not sure why this has been changed 🤔 hasShownSavePrompt
should stay as it was and with showSavePrompt
defaulting to false these tests should work without change unless I'm missing something
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I guess it's because we're now defaulting to not showing the save prompt, so the change on line 109 of WebComponentLoader
means that we're passing this in as true
to stop the prompt from showing unless they've specified that they want it... Maybe the naming is not the best though?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
AH Sorry!! I thought this was setting state, I should've checked the few lines above. Yea absolutely, that's fine
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
closes #953