{"push_id":701361302,"size":7,"distinct_size":1,"ref":"refs/heads/review-together/release-1.0.x","head":"bdd320bdea86981c8858ff95bdb3567cf520f066","before":"e24a93941562b28739a06a97db9509b391a2d7a2","commits":[{"sha":"987e36872fcfb1c8af79141e1db9bdc6f6751d8a","author":{"email":"2314b2e3a4a1f7db165be2aafbf1efd78f28cc97@beanbaginc.com","name":"Christian Hammond"},"message":"Add branch information to .reviewboardrc.","distinct":false,"url":"https://api.github.com/repos/reviewboard/rb-extension-pack/commits/987e36872fcfb1c8af79141e1db9bdc6f6751d8a"},{"sha":"6981472b3f3a781c9e78bf8a3e1ee6593ffdfbab","author":{"email":"2314b2e3a4a1f7db165be2aafbf1efd78f28cc97@beanbaginc.com","name":"Christian Hammond"},"message":"[rbdemo] Delete the SCMTools prior to loading in fixtures.\n\nIt was easy to get into a situation where SCMTools from the fixtures\nconflicted with what was populated from registerscmtools. We now clear\nout the SCMTools in the database immediately before we load the fixture\ndata.\n\nTesting Done:\nThis fixed a problem I had on demo.reviewboard.org with deploying updates\nto the fixtures.\n\nReviewed at https://reviews.reviewboard.org/r/7138/","distinct":false,"url":"https://api.github.com/repos/reviewboard/rb-extension-pack/commits/6981472b3f3a781c9e78bf8a3e1ee6593ffdfbab"},{"sha":"743ada6c9de383af9de765eab041b90bca188084","author":{"email":"bacce4698031de677ea8dbee4856ef495373d149@gmail.com","name":"Wang Jun Sun"},"message":"[checklist] Improve checklist extension and user interface\n\nThis change aims to polish up the existing checklist UI as well as its code base.\n\nOn the UI:\n\n+ Shifted the UI to be **fixed at the bottom** (better than in the middle of\n  nowhere).\n+ Uses existing Reviewboard icons as **buttons** (versus textual buttons).\n+ Reworked toggling of the interface (using **CSS animation**).\n+ Editing an item selects/highlights the entire text (like renaming files).\n+ Ability to cancel edit via Esc key or button.\n+ **Word wrapping** of long item descriptions.\n+ Now **escapes special characters** in the item descriptions.\n+ Trims the string before adding or editing, avoids empty descriptions.\n\nIn the front end:\n\n+ Removed *delete checklist* functionality (since new checklist is generated\n  upon refresh).\n+ Moved *remove item* method from collection view to item view.\n+ So that item ids do not have to be *stored in DOM*.\n+ Fixed a bug where editing multiple items only picks the value of the first.\n  item being edited.\n+ Enforce hyphens for ids and classes, and single quotes for strings.\n\nTesting Done:\nGlass box front end testing on Chrome 40.0.2214. Quick tests for UI on Safari\n8.0.2.\n\n+ Toggling size works\n+ Marking an item as done or undone works\n+ Deleting an item works\n+ Editing an item works\n+ Cancelling item editing works\n+ Adding an item works\n+ Long text is wrapped\n+ Special characters are escaped\n+ Can edit multiple items without conflict\n+ Whitespace in description is trimmed\n\nReviewed at https://reviews.reviewboard.org/r/6969/","distinct":false,"url":"https://api.github.com/repos/reviewboard/rb-extension-pack/commits/743ada6c9de383af9de765eab041b90bca188084"},{"sha":"b747249629db57f7e123af428a2f81a803f823b9","author":{"email":"bacce4698031de677ea8dbee4856ef495373d149@gmail.com","name":"Wang Jun Sun"},"message":"[checklist] Implement checklist templates in account pages\n\nAn additional functionality of the checklist extension is to allow users to\ncreate individual templates which can be quickly added to any existing\nchecklists. This change does not enable checklist templates to be imported\ninto existing checklists.\n\nChecklist templates are stored with two attributes, their title as a string and\ntheir list of items in JSON format.\n\nThe form for users to modify their checklist templates is under their account\npreferences page. In the form, users can add checklist templates, remove\nchecklist templates, or edit an existing checklist template.\n\nThe form will automatically add new item fields when it detects that the user\nis editing the last item field, or when the user presses enter.\n\nTesting Done:\nOn Chrome 41.0.2272 and Safari 8.0.2:\n\n- Basic CRUD functionality - adding, editing and removing - works as it should.\n- Does not accept templates without a title.\n- Empty items are not included upon form submission.\n- Tested and ensured that the UI shortcuts (enter, tab) work.\n- Verified that the issue where the menu entry will disappear has been resolved\n  (after #7103).\n- Ensured that users can only view and edit their own templates.\n\nReviewed at https://reviews.reviewboard.org/r/7099/","distinct":false,"url":"https://api.github.com/repos/reviewboard/rb-extension-pack/commits/b747249629db57f7e123af428a2f81a803f823b9"},{"sha":"27afa7845359e8301db79257891f7dd53c95dbd6","author":{"email":"bacce4698031de677ea8dbee4856ef495373d149@gmail.com","name":"Wang Jun Sun"},"message":"[checklist] Refactoring of checklist\n\nRefactoring includes\n\n- Use of `request.user` instead of passing `user_id` in every HTTP request\n- Updating of the view only after successful API response\n- Renamed status of each item from `finished` to `checked`\n- Use of `ChecklistItem` non-database resource API instead of using HTTP\n  request parameters\n- Moving of front-end API logic from view to model\n- CSS hyphens everywhere instead of underscores\n- Improving Python code (renaming variables, removing redundant code)\n- Start checklist item IDs from 0 instead of 1.\n\nTesting done:\nBasic CRUD functionality remains. Ensured that the views will not update unless\nthere is a successful response from the server.\n\nReviewed at https://reviews.reviewboard.org/r/7176/","distinct":false,"url":"https://api.github.com/repos/reviewboard/rb-extension-pack/commits/27afa7845359e8301db79257891f7dd53c95dbd6"},{"sha":"b635ba7cac957c0ae19e96c035b5d46927e64290","author":{"email":"c4a40333d2e70b3fcac4e7cd05d0e9037471cc26@gmail.com","name":"David Trowbridge"},"message":"Apply some fixes to review_together.\n\nThis change was partially sent to us via a pull request. The major fixes remove\nthe 'var' definition for ReviewTogetherJS and fix the setup.py file to use the\nReview Board extension setup method instead of setuptools. The rest of the\nfixes here are purely cosmetic.","distinct":false,"url":"https://api.github.com/repos/reviewboard/rb-extension-pack/commits/b635ba7cac957c0ae19e96c035b5d46927e64290"},{"sha":"bdd320bdea86981c8858ff95bdb3567cf520f066","author":{"email":"a737e13205c91e40c7c65a95eda2a9d9c2c34899@gmail.com","name":"Mike Conley"},"message":"[review-together] Bump version to 1.0.0b.","distinct":true,"url":"https://api.github.com/repos/reviewboard/rb-extension-pack/commits/bdd320bdea86981c8858ff95bdb3567cf520f066"}]}