GOOGLE-145 Duplicating a site also duplicates Google calendar association

Activity

CONTRIB-34 10

Keyboard shortcuts  
Summarize the review outcomes (optional)
 
#permalink

Details

Warning: no files are visible, they have all been filtered.
Participant Role Time Spent Comments Latest Comment
Author 5m 4 Added braces around the return. Yes, I always like the br...
Reviewer - Complete 15m 5 Looks nice and clean and simple.
Reviewer - Complete 3m 1 All looks good with Dave's suggested changes
Total   23m 10  
#permalink

Objectives

86533: GOOGLE-145 Duplicating a site also duplicates Google calendar association. Created an observer to watch for the end of a site duplication event and to remove the gcalid property from the newly created site.

Branches in review

#permalink

Issues Raised From Comments

Key Summary State Assignee
#permalink

General Comments

David Haines

Looks nice and clean and simple.

Looks nice and clean and simple.

Beth Kirschner

All looks good with Dave's suggested changes

All looks good with Dave's suggested changes

/umich/google/.../impl/GcalEventWatcher.java Added 8
Open in IDE #permalink
/umich/google/.../WEB-INF/components.xml Changed
Open in IDE #permalink

Review updated: Reload | Ignore | Collapse

You cannot reload the review while writing a comment.

Create issue

X
Assign To Me

Log time against