Skip to content

fix: replace Blockly.selected to Blockly.getSelected() - #1661

Merged
BeksOmega merged 3 commits into
RaspberryPiFoundation:masterfrom
mrkprdo:fix-cross-tab-copy-paste
Apr 11, 2023
Merged

fix: replace Blockly.selected to Blockly.getSelected()#1661
BeksOmega merged 3 commits into
RaspberryPiFoundation:masterfrom
mrkprdo:fix-cross-tab-copy-paste

Conversation

@mrkprdo

@mrkprdo mrkprdo commented Apr 11, 2023

Copy link
Copy Markdown
Contributor

Some warnings showing Blockly.selected will be removed soon. So needed to have this updated here.

@mrkprdo
mrkprdo requested a review from a team as a code owner April 11, 2023 15:52
@mrkprdo
mrkprdo requested review from cpcallen and removed request for a team April 11, 2023 15:52
@google-cla

google-cla Bot commented Apr 11, 2023

Copy link
Copy Markdown

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@mrkprdo mrkprdo changed the title replace Blockly.selected to Blockly.getSelected() fix: replace Blockly.selected to Blockly.getSelected() Apr 11, 2023
@BeksOmega
BeksOmega requested review from BeksOmega and removed request for cpcallen April 11, 2023 16:21
@BeksOmega BeksOmega assigned BeksOmega and unassigned cpcallen Apr 11, 2023

@BeksOmega BeksOmega left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks great! If you sign the CLA and CI passes this should be gtg. Thank you so muchfor the fix =)

@mrkprdo mrkprdo closed this Apr 11, 2023
@mrkprdo mrkprdo reopened this Apr 11, 2023
@mrkprdo

mrkprdo commented Apr 11, 2023

Copy link
Copy Markdown
Contributor Author

@BeksOmega i don't think I can merge? lol

@BeksOmega

Copy link
Copy Markdown
Contributor

@BeksOmega i don't think I can merge? lol

Yep, I gotchu. Thank you again for the fix! We really appreciate it =)

@BeksOmega
BeksOmega merged commit 38c4504 into RaspberryPiFoundation:master Apr 11, 2023
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.

3 participants