GitHub PullRequestEvent at 2014-03-03 05:52:44
Repository: declarative-lookup-rollup-summaries
Actor: wes1278
Payload: {"action":"opened","number":15,"pull_request":{"url":"https://api.github.com/repos/afawcett/declarative-lookup-rollup-summaries/pulls/15","id":13101641,"html_url":"https://github.com/afawcett/declarative-lookup-rollup-summaries/pull/15","diff_url":"https://github.com/afawcett/declarative-lookup-rollup-summaries/pull/15.diff","patch_url":"https://github.com/afawcett/declarative-lookup-rollup-summaries/pull/15.patch","issue_url":"https://api.github.com/repos/afawcett/declarative-lookup-rollup-summaries/issues/15","number":15,"state":"open","title":"rollup summary new page v1.0","user":{"login":"wes1278","id":596315,"avatar_url":"https://avatars.githubusercontent.com/u/596315","gravatar_id":"117514d9787b8b384eb01adc74ea9f50","url":"https://api.github.com/users/wes1278","html_url":"https://github.com/wes1278","followers_url":"https://api.github.com/users/wes1278/followers","following_url":"https://api.github.com/users/wes1278/following{/other_user}","gists_url":"https://api.github.com/users/wes1278/gists{/gist_id}","starred_url":"https://api.github.com/users/wes1278/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/wes1278/subscriptions","organizations_url":"https://api.github.com/users/wes1278/orgs","repos_url":"https://api.github.com/users/wes1278/repos","events_url":"https://api.github.com/users/wes1278/events{/privacy}","received_events_url":"https://api.github.com/users/wes1278/received_events","type":"User","site_admin":false},"body":"Hey Andrew,\r\n\r\nI whipped together a pretty simple page to help the creation of these lookup rollup summary records. Here are some features:\r\n- [x] Select Parent Object from selectlist (filtered by anything updateable & createable,.)\r\n- [x] select parent aggregate result from select list (filterd by anything accessible, updateable, and numeric/date)\r\n- [x] select child record (filtered by what actually has a lookup to the parent you chose)\r\n- [x] no need to type the relationship field, it's displayed for you when you're choosing the child object.\r\n- [x] select child field to aggregate from select list (filtered by anything accessible, updateable, and numeric/date)\r\n\r\nThere are a few things that I was hoping you might help be able to help me with.\r\n- [ ] Routing to the trigger deployment method (if that's possible) so that the save event can deploy the trigger if it's not present. Right now, I have the check logic in there, but I couldn't make sense of how this trigger creation was happening in the time I had. Check the \"save\" method in the page controller.\r\n- [ ] See if there is anything you see that would \"fit\" into your enterprise pattern any better than I've done. I'd love to get some feedback.\r\n\r\nAlthough the page is usable right now, here is a quick list of the things I'm hoping to accomplish still with this page. Let me know if you have any feedback about my ideas.\r\n- [ ] Decide if this page should also override the edit page or if there should be a different page.\r\n- [ ] Maybe use jquery.autocomplete or something similar instead of select lists. (js remoting)\r\n- [ ] maybe use a \"step/wizard\" like approach instead of shoving it all on 1 entry page. (I have a quick mini visualforce framework for these simple wizards)\r\n- [ ] Create a wrapper class for criteria.\r\n- [ ] create a parsing method to turn a where clause into a list of wrapper class objects.\r\n- [ ] create a creation method to turn a list of wrapper class objects into a where clause (maybe soql builder)?\r\n- [ ] use some javascript/jquery to add/remove/edit wrapper objects representing criteria.\r\n- [ ] add an input for filter logic ie 1 AND (2 or 3)\r\n\r\nOn an unrelated note, do you happen to have a repo setup to demonstrate the basis for your apex enterprise pattern? I've attended your sessions at Dreamforce and tried to rip this repo apart but I think I've biten off a little more than I can chew. I was hoping you might have a repo that had your base classes, and like a single simple example class.\r\n\r\nOn another unrelated note (sorry for so many questions), I was wondering if you use MavensMate for development? If you do, would you mind telling me how you deal with keeping your mavensmate workspace/folder working with git. I spent at least twice as much trying to get git and mavensmate to work together compared to how much time I spent on the page. In your repo's you have the src folder inside another folder. I couldn't get mavensmate to actually let me use that sub folder so I had to move things around and try to convince git to do what I wanted. (considering I'm still pretty new to git, that was also a challenge. :)\r\n\r\nThanks!\r\n![dlrs](https://f.cloud.github.com/assets/596315/2306785/a3a04bac-a296-11e3-9252-7ae265fc5716.png)","created_at":"2014-03-03T05:52:44Z","updated_at":"2014-03-03T05:52:44Z","closed_at":null,"merged_at":null,"merge_commit_sha":null,"assignee":null,"milestone":null,"commits_url":"https://api.github.com/repos/afawcett/declarative-lookup-rollup-summaries/pulls/15/commits","review_comments_url":"https://api.github.com/repos/afawcett/declarative-lookup-rollup-summaries/pulls/15/comments","review_comment_url":"https://api.github.com/repos/afawcett/declarative-lookup-rollup-summaries/pulls/comments/{number}","comments_url":"https://api.github.com/repos/afawcett/declarative-lookup-rollup-summaries/issues/15/comments","statuses_url":"https://api.github.com/repos/afawcett/declarative-lookup-rollup-summaries/statuses/b134aaabd6bb918c771faab18fd323a2a04fb877","head":{"label":"wes1278:master","ref":"master","sha":"b134aaabd6bb918c771faab18fd323a2a04fb877","user":{"login":"wes1278","id":596315,"avatar_url":"https://avatars.githubusercontent.com/u/596315","gravatar_id":"117514d9787b8b384eb01adc74ea9f50","url":"https://api.github.com/users/wes1278","html_url":"https://github.com/wes1278","followers_url":"https://api.github.com/users/wes1278/followers","following_url":"https://api.github.com/users/wes1278/following{/other_user}","gists_url":"https://api.github.com/users/wes1278/gists{/gist_id}","starred_url":"https://api.github.com/users/wes1278/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/wes1278/subscriptions","organizations_url":"https://api.github.com/users/wes1278/orgs","repos_url":"https://api.github.com/users/wes1278/repos","events_url":"https://api.github.com/users/wes1278/events{/privacy}","received_events_url":"https://api.github.com/users/wes1278/received_events","type":"User","site_admin":false},"repo":{"id":17305516,"name":"declarative-lookup-rollup-summaries","full_name":"wes1278/declarative-lookup-rollup-summaries","owner":{"login":"wes1278","id":596315,"avatar_url":"https://avatars.githubusercontent.com/u/596315","gravatar_id":"117514d9787b8b384eb01adc74ea9f50","url":"https://api.github.com/users/wes1278","html_url":"https://github.com/wes1278","followers_url":"https://api.github.com/users/wes1278/followers","following_url":"https://api.github.com/users/wes1278/following{/other_user}","gists_url":"https://api.github.com/users/wes1278/gists{/gist_id}","starred_url":"https://api.github.com/users/wes1278/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/wes1278/subscriptions","organizations_url":"https://api.github.com/users/wes1278/orgs","repos_url":"https://api.github.com/users/wes1278/repos","events_url":"https://api.github.com/users/wes1278/events{/privacy}","received_events_url":"https://api.github.com/users/wes1278/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/wes1278/declarative-lookup-rollup-summaries","description":"Declarative Rollup Summaries for Lookups","fork":true,"url":"https://api.github.com/repos/wes1278/declarative-lookup-rollup-summaries","forks_url":"https://api.github.com/repos/wes1278/declarative-lookup-rollup-summaries/forks","keys_url":"https://api.github.com/repos/wes1278/declarative-lookup-rollup-summaries/keys{/key_id}","collaborators_url":"https://api.github.com/repos/wes1278/declarative-lookup-rollup-summaries/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/wes1278/declarative-lookup-rollup-summaries/teams","hooks_url":"https://api.github.com/repos/wes1278/declarative-lookup-rollup-summaries/hooks","issue_events_url":"https://api.github.com/repos/wes1278/declarative-lookup-rollup-summaries/issues/events{/number}","events_url":"https://api.github.com/repos/wes1278/declarative-lookup-rollup-summaries/events","assignees_url":"https://api.github.com/repos/wes1278/declarative-lookup-rollup-summaries/assignees{/user}","branches_url":"https://api.github.com/repos/wes1278/declarative-lookup-rollup-summaries/branches{/branch}","tags_url":"https://api.github.com/repos/wes1278/declarative-lookup-rollup-summaries/tags","blobs_url":"https://api.github.com/repos/wes1278/declarative-lookup-rollup-summaries/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/wes1278/declarative-lookup-rollup-summaries/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/wes1278/declarative-lookup-rollup-summaries/git/refs{/sha}","trees_url":"https://api.github.com/repos/wes1278/declarative-lookup-rollup-summaries/git/trees{/sha}","statuses_url":"https://api.github.com/repos/wes1278/declarative-lookup-rollup-summaries/statuses/{sha}","languages_url":"https://api.github.com/repos/wes1278/declarative-lookup-rollup-summaries/languages","stargazers_url":"https://api.github.com/repos/wes1278/declarative-lookup-rollup-summaries/stargazers","contributors_url":"https://api.github.com/repos/wes1278/declarative-lookup-rollup-summaries/contributors","subscribers_url":"https://api.github.com/repos/wes1278/declarative-lookup-rollup-summaries/subscribers","subscription_url":"https://api.github.com/repos/wes1278/declarative-lookup-rollup-summaries/subscription","commits_url":"https://api.github.com/repos/wes1278/declarative-lookup-rollup-summaries/commits{/sha}","git_commits_url":"https://api.github.com/repos/wes1278/declarative-lookup-rollup-summaries/git/commits{/sha}","comments_url":"https://api.github.com/repos/wes1278/declarative-lookup-rollup-summaries/comments{/number}","issue_comment_url":"https://api.github.com/repos/wes1278/declarative-lookup-rollup-summaries/issues/comments/{number}","contents_url":"https://api.github.com/repos/wes1278/declarative-lookup-rollup-summaries/contents/{+path}","compare_url":"https://api.github.com/repos/wes1278/declarative-lookup-rollup-summaries/compare/{base}...{head}","merges_url":"https://api.github.com/repos/wes1278/declarative-lookup-rollup-summaries/merges","archive_url":"https://api.github.com/repos/wes1278/declarative-lookup-rollup-summaries/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/wes1278/declarative-lookup-rollup-summaries/downloads","issues_url":"https://api.github.com/repos/wes1278/declarative-lookup-rollup-summaries/issues{/number}","pulls_url":"https://api.github.com/repos/wes1278/declarative-lookup-rollup-summaries/pulls{/number}","milestones_url":"https://api.github.com/repos/wes1278/declarative-lookup-rollup-summaries/milestones{/number}","notifications_url":"https://api.github.com/repos/wes1278/declarative-lookup-rollup-summaries/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/wes1278/declarative-lookup-rollup-summaries/labels{/name}","releases_url":"https://api.github.com/repos/wes1278/declarative-lookup-rollup-summaries/releases{/id}","created_at":"2014-03-01T02:02:01Z","updated_at":"2014-03-03T05:52:44Z","pushed_at":"2014-03-03T05:08:13Z","git_url":"git://github.com/wes1278/declarative-lookup-rollup-summaries.git","ssh_url":"git@github.com:wes1278/declarative-lookup-rollup-summaries.git","clone_url":"https://github.com/wes1278/declarative-lookup-rollup-summaries.git","svn_url":"https://github.com/wes1278/declarative-lookup-rollup-summaries","homepage":null,"size":312,"stargazers_count":0,"watchers_count":0,"language":"Apex","has_issues":false,"has_downloads":true,"has_wiki":true,"forks_count":0,"mirror_url":null,"open_issues_count":0,"forks":0,"open_issues":0,"watchers":0,"default_branch":"master","master_branch":"master"}},"base":{"label":"afawcett:master","ref":"master","sha":"dc739898e502f70664de2d8fcd8a42bc721728fa","user":{"login":"afawcett","id":1167760,"avatar_url":"https://avatars.githubusercontent.com/u/1167760","gravatar_id":"a2b71aad1874922e580b17e93c2495c5","url":"https://api.github.com/users/afawcett","html_url":"https://github.com/afawcett","followers_url":"https://api.github.com/users/afawcett/followers","following_url":"https://api.github.com/users/afawcett/following{/other_user}","gists_url":"https://api.github.com/users/afawcett/gists{/gist_id}","starred_url":"https://api.github.com/users/afawcett/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/afawcett/subscriptions","organizations_url":"https://api.github.com/users/afawcett/orgs","repos_url":"https://api.github.com/users/afawcett/repos","events_url":"https://api.github.com/users/afawcett/events{/privacy}","received_events_url":"https://api.github.com/users/afawcett/received_events","type":"User","site_admin":false},"repo":{"id":11242179,"name":"declarative-lookup-rollup-summaries","full_name":"afawcett/declarative-lookup-rollup-summaries","owner":{"login":"afawcett","id":1167760,"avatar_url":"https://avatars.githubusercontent.com/u/1167760","gravatar_id":"a2b71aad1874922e580b17e93c2495c5","url":"https://api.github.com/users/afawcett","html_url":"https://github.com/afawcett","followers_url":"https://api.github.com/users/afawcett/followers","following_url":"https://api.github.com/users/afawcett/following{/other_user}","gists_url":"https://api.github.com/users/afawcett/gists{/gist_id}","starred_url":"https://api.github.com/users/afawcett/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/afawcett/subscriptions","organizations_url":"https://api.github.com/users/afawcett/orgs","repos_url":"https://api.github.com/users/afawcett/repos","events_url":"https://api.github.com/users/afawcett/events{/privacy}","received_events_url":"https://api.github.com/users/afawcett/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/afawcett/declarative-lookup-rollup-summaries","description":"Declarative Rollup Summaries for Lookups","fork":false,"url":"https://api.github.com/repos/afawcett/declarative-lookup-rollup-summaries","forks_url":"https://api.github.com/repos/afawcett/declarative-lookup-rollup-summaries/forks","keys_url":"https://api.github.com/repos/afawcett/declarative-lookup-rollup-summaries/keys{/key_id}","collaborators_url":"https://api.github.com/repos/afawcett/declarative-lookup-rollup-summaries/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/afawcett/declarative-lookup-rollup-summaries/teams","hooks_url":"https://api.github.com/repos/afawcett/declarative-lookup-rollup-summaries/hooks","issue_events_url":"https://api.github.com/repos/afawcett/declarative-lookup-rollup-summaries/issues/events{/number}","events_url":"https://api.github.com/repos/afawcett/declarative-lookup-rollup-summaries/events","assignees_url":"https://api.github.com/repos/afawcett/declarative-lookup-rollup-summaries/assignees{/user}","branches_url":"https://api.github.com/repos/afawcett/declarative-lookup-rollup-summaries/branches{/branch}","tags_url":"https://api.github.com/repos/afawcett/declarative-lookup-rollup-summaries/tags","blobs_url":"https://api.github.com/repos/afawcett/declarative-lookup-rollup-summaries/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/afawcett/declarative-lookup-rollup-summaries/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/afawcett/declarative-lookup-rollup-summaries/git/refs{/sha}","trees_url":"https://api.github.com/repos/afawcett/declarative-lookup-rollup-summaries/git/trees{/sha}","statuses_url":"https://api.github.com/repos/afawcett/declarative-lookup-rollup-summaries/statuses/{sha}","languages_url":"https://api.github.com/repos/afawcett/declarative-lookup-rollup-summaries/languages","stargazers_url":"https://api.github.com/repos/afawcett/declarative-lookup-rollup-summaries/stargazers","contributors_url":"https://api.github.com/repos/afawcett/declarative-lookup-rollup-summaries/contributors","subscribers_url":"https://api.github.com/repos/afawcett/declarative-lookup-rollup-summaries/subscribers","subscription_url":"https://api.github.com/repos/afawcett/declarative-lookup-rollup-summaries/subscription","commits_url":"https://api.github.com/repos/afawcett/declarative-lookup-rollup-summaries/commits{/sha}","git_commits_url":"https://api.github.com/repos/afawcett/declarative-lookup-rollup-summaries/git/commits{/sha}","comments_url":"https://api.github.com/repos/afawcett/declarative-lookup-rollup-summaries/comments{/number}","issue_comment_url":"https://api.github.com/repos/afawcett/declarative-lookup-rollup-summaries/issues/comments/{number}","contents_url":"https://api.github.com/repos/afawcett/declarative-lookup-rollup-summaries/contents/{+path}","compare_url":"https://api.github.com/repos/afawcett/declarative-lookup-rollup-summaries/compare/{base}...{head}","merges_url":"https://api.github.com/repos/afawcett/declarative-lookup-rollup-summaries/merges","archive_url":"https://api.github.com/repos/afawcett/declarative-lookup-rollup-summaries/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/afawcett/declarative-lookup-rollup-summaries/downloads","issues_url":"https://api.github.com/repos/afawcett/declarative-lookup-rollup-summaries/issues{/number}","pulls_url":"https://api.github.com/repos/afawcett/declarative-lookup-rollup-summaries/pulls{/number}","milestones_url":"https://api.github.com/repos/afawcett/declarative-lookup-rollup-summaries/milestones{/number}","notifications_url":"https://api.github.com/repos/afawcett/declarative-lookup-rollup-summaries/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/afawcett/declarative-lookup-rollup-summaries/labels{/name}","releases_url":"https://api.github.com/repos/afawcett/declarative-lookup-rollup-summaries/releases{/id}","created_at":"2013-07-07T23:48:42Z","updated_at":"2014-02-11T02:20:07Z","pushed_at":"2014-02-11T02:20:06Z","git_url":"git://github.com/afawcett/declarative-lookup-rollup-summaries.git","ssh_url":"git@github.com:afawcett/declarative-lookup-rollup-summaries.git","clone_url":"https://github.com/afawcett/declarative-lookup-rollup-summaries.git","svn_url":"https://github.com/afawcett/declarative-lookup-rollup-summaries","homepage":null,"size":498,"stargazers_count":14,"watchers_count":14,"language":"Apex","has_issues":true,"has_downloads":true,"has_wiki":true,"forks_count":6,"mirror_url":null,"open_issues_count":4,"forks":6,"open_issues":4,"watchers":14,"default_branch":"master","master_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/afawcett/declarative-lookup-rollup-summaries/pulls/15"},"html":{"href":"https://github.com/afawcett/declarative-lookup-rollup-summaries/pull/15"},"issue":{"href":"https://api.github.com/repos/afawcett/declarative-lookup-rollup-summaries/issues/15"},"comments":{"href":"https://api.github.com/repos/afawcett/declarative-lookup-rollup-summaries/issues/15/comments"},"review_comments":{"href":"https://api.github.com/repos/afawcett/declarative-lookup-rollup-summaries/pulls/15/comments"},"review_comment":{"href":"https://api.github.com/repos/afawcett/declarative-lookup-rollup-summaries/pulls/comments/{number}"},"commits":{"href":"https://api.github.com/repos/afawcett/declarative-lookup-rollup-summaries/pulls/15/commits"},"statuses":{"href":"https://api.github.com/repos/afawcett/declarative-lookup-rollup-summaries/statuses/b134aaabd6bb918c771faab18fd323a2a04fb877"}},"merged":false,"mergeable":null,"mergeable_state":"unknown","merged_by":null,"comments":0,"review_comments":0,"commits":5,"additions":260,"deletions":0,"changed_files":4}}