-
Notifications
You must be signed in to change notification settings - Fork 64
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
Add warning modal dialog #1390
Add warning modal dialog #1390
Conversation
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 think we may have a bug that affects this PR.
Either that, or we're missing something.
If I recall correctly, Community and Design (@Pickysaurus @captainsandypants) wanted for the dialog to show on every boot. And later down the road in 'Beta' we'd add a don't show again
option.
That is actually the case right now, but I'm not sure if that's happening for the right reason.
In AlphaWarningViewModel.MaybeShow
we try to obtain a previous instance of AlphaSettings
from MnemonicDB, that is _settingsManager.Get<AlphaSettings>()
. On my machine (with fresh DataModel), this seems to always return an AlphaSettings
where HasShownModal
is false. Even though in the previous run of the App we've set that to true.
Switching to the JSON backend seems to correct this behaviour.
public static ISettingsBuilder Configure(ISettingsBuilder settingsBuilder)
{
return settingsBuilder
.ConfigureStorageBackend<AlphaSettings>(builder => builder.UseJson());
}
In which case the dialog is not shown again (we get HasShownModal
== true on subsequent boot).
Can you doublecheck that what I'm experiencing here is a bug with the MMDB settings backend?
Oh and @captainsandypants ; currently the |
<TextBlock Classes="BodyLGBold">Important</TextBlock> | ||
</StackPanel> | ||
|
||
<TextBlock Classes="BodyMDNormal" TextWrapping="Wrap">This app is still very early in development and currently only supports Stardew Valley. Please ensure no mods are installed in your game folder before testing.</TextBlock> |
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.
Technically speaking these should be localized, given we will likely reuse this dialog for beta; which may have non-English people trying to participate, but it's not big enough a deal to block the PR. Am just commenting as it's worth pointing out.
@Sewer56 can you explain your comment, I'm not sure I follow. The overlay will only appear once, similar to the telemetry dialog. |
And I just realized I forgot to hook up the help icon. |
The tl;dr is I reset my DataStore to get to a clean state and
I was asking if this behaviour is only something that reproduces on my machine. |
Sounds like a you problem 😅. |
Goddamn it 😅 |
Ah, a full rebuild fixed it. Very strange. |
Resolves #1376.