Skip to content

Conversation

@benjiwheeler
Copy link
Contributor

Resolves

Fix to #4700

Proposed Changes

Does NOT show user a "Replace contents of the current project?" modal if they are logged in, viewing someone else's project, and do File->Load from your computer.

This restores current functionality, rather than introducing a change to it like #4700 did.

Reason for Changes

We found this message very confusing, since your upload does NOT replace the project at the id you're looking at when you do File->Load from your computer. (And it shouldn't!)

Copy link
Contributor

@paulkaplan paulkaplan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@benjiwheeler benjiwheeler merged commit d37f1fc into scratchfoundation:develop Apr 18, 2019
@benjiwheeler benjiwheeler deleted the fix-upload-confirm branch April 18, 2019 15:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants