rhtyd commented on issue #99: [WIP] - Recurring Volume Snapshot Custom Action Form URL: https://github.com/apache/cloudstack-primate/pull/99#issuecomment-574046988 @utchoang reviewed, LGTM except for this few issues: 1. Please always checkout new work/branch from origin/master, otherwise the PRs will have commits from other PRs/branches and will cause difficulties in reviewing them. I've fixed it manually for at least two PRs for now. 2. When I open the recurring action form the first time, I'm unable to list the timezones:  I've moved the timezone file/folder under utils and changed the import paths. It's also okay if instead of resolve/importing the json, you want to consume it as a static object/map. 3. While you're at it, can you also fix the createSnapshot form which takes in tags, which is not supported out of the box: 
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
