Skip to content

Nit about method name - #109

Merged
sqrrrl merged 1 commit into
masterfrom
asrivas-patch-3
Jul 17, 2019
Merged

Nit about method name#109
sqrrrl merged 1 commit into
masterfrom
asrivas-patch-3

Conversation

@asrivas

@asrivas asrivas commented Apr 3, 2019

Copy link
Copy Markdown
Contributor

No description provided.

grant
grant previously approved these changes Apr 4, 2019

@trungst368 trungst368 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

SugGesting changes

* with the speaker notes for each slide.
*/
function generateSlideScript() {
function generateSlidesScript() {

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Need to update the function name in onOpen too.

@sqrrrl
sqrrrl dismissed grant’s stale review May 8, 2019 19:34

Missing required changes for script to still work

oshliaer added a commit to oshliaer/apps-script-samples that referenced this pull request Jul 17, 2019
@sqrrrl
sqrrrl merged commit 5c22bb6 into master Jul 17, 2019
sqrrrl added a commit that referenced this pull request Jul 17, 2019
brentchang pushed a commit to brentchang/apps-script-samples that referenced this pull request Dec 3, 2023
brentchang pushed a commit to brentchang/apps-script-samples that referenced this pull request Dec 3, 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.

4 participants