Skip to content

Comments

Remove duplicate computer beeps#1401

Merged
paulkaplan merged 1 commit intoscratchfoundation:masterfrom
kyleplo:master
Feb 7, 2018
Merged

Remove duplicate computer beeps#1401
paulkaplan merged 1 commit intoscratchfoundation:masterfrom
kyleplo:master

Conversation

@kyleplo
Copy link
Contributor

@kyleplo kyleplo commented Feb 2, 2018

Resolves

#1355

Proposed Changes

Removes the duplicate computer beeps sound

Reason for Changes

You shouldn't have two of the same sounds

Test Coverage

If you search for "computer beep", there are 3 results, but only 2 different sounds

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.

Looks good

@paulkaplan paulkaplan merged commit 8134caa into scratchfoundation:master Feb 7, 2018
@paulkaplan
Copy link
Contributor

paulkaplan commented Feb 7, 2018

Sorry @kyleplo I messed up on this and didn't see that it was not linked to a help-wanted issue. As mentioned in #1356, which was identical to this PR,

Thanks, but this should be resolved in the source project that JSON is generated from. Please refrain from submitting PRs for issues that are not labeled as “help wanted”.
I'm going to revert this until we can change those source project files.

I know this looks like a small change, but it is important to not submit PRs for issues that haven't been confirmed "help-wanted" because of things like this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants