Skip to content

UI string review for google docs and calendar - #2

Merged
mhoeber merged 1 commit into
masterfrom
markhoeber-studio-UI-review
Nov 24, 2014
Merged

UI string review for google docs and calendar#2
mhoeber merged 1 commit into
masterfrom
markhoeber-studio-UI-review

Conversation

@mhoeber

@mhoeber mhoeber commented Nov 21, 2014

Copy link
Copy Markdown
Contributor

DOC-1294

@mhoeber

mhoeber commented Nov 21, 2014

Copy link
Copy Markdown
Contributor Author

@martynjames

also should tag Carol Tong and Marco; they don't appear to be in edx-solutions yet

@mhoeber

mhoeber commented Nov 21, 2014

Copy link
Copy Markdown
Contributor Author

@catong

Comment thread google_drive/google_calendar.py Outdated

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

It might be helpful to say something like "In your source Google calendar, open Settings and copy..."

@mhoeber

mhoeber commented Nov 24, 2014

Copy link
Copy Markdown
Contributor Author

@catong updated per comments. @martynjames can you confirm.

Carol, I should know but am not sure how to build a sandbox with this xblock added. If we can get it merged and have an updated sandbox, we should have time to review then.

@martynjames

Copy link
Copy Markdown

Thanks @mhoeber - LGTM 👍 When merged, I'll rebuild the sandbox with the changes so that you can see them

@catong

catong commented Nov 24, 2014

Copy link
Copy Markdown

👍

@mhoeber
mhoeber force-pushed the markhoeber-studio-UI-review branch from 1621944 to 366b3c9 Compare November 24, 2014 15:03
mhoeber added a commit that referenced this pull request Nov 24, 2014
UI string review for google docs and calendar
@mhoeber
mhoeber merged commit 5f3c329 into master Nov 24, 2014
@mhoeber
mhoeber deleted the markhoeber-studio-UI-review branch November 24, 2014 15:03
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