{"push_id":2445509247,"size":303,"distinct_size":1,"ref":"refs/heads/bbb-feature-recording-ready-extension","head":"ea9ab88fb5fce25196fa91993f00fe1af04e1937","before":"d9d2fa57b2ba525b987d66e26e38cf4800ec77cf","commits":[{"sha":"1b43c898dee33d76b68cf19619ea0d596d5da9ae","author":{"name":"Ryan Shaw","email":"8f18e670ec432ee5e15365d38d203f741b57c02e@instructure.com"},"message":"Covert usageRightsSelectBox from coffee to js\n\nChange-Id: I93dbf3097464e2625978b00850c6dc7123c977fd\nReviewed-on: https://gerrit.instructure.com/143361\nReviewed-by: Brent Burgoyne <bburgoyne@instructure.com>\nTested-by: Jenkins\nProduct-Review: Ryan Shaw <ryan@instructure.com>\nQA-Review: Ryan Shaw <ryan@instructure.com>","distinct":false,"url":"https://api.github.com/repos/jfederico/canvas-lms/commits/1b43c898dee33d76b68cf19619ea0d596d5da9ae"},{"sha":"ca30cb3035401f8142b9d428e35341eb21caf8f4","author":{"name":"Ryan Shaw","email":"8f18e670ec432ee5e15365d38d203f741b57c02e@instructure.com"},"message":"Don’t include all react_files strings in usageRights\n\nTest plan:\n* The strings for react_files should not be included in the webpack\ncommon bundle in production\n* the usage rights selector should still work\n\nChange-Id: I6a578c42e8b1a2edaec5de56fa921312d6292d1e\nReviewed-on: https://gerrit.instructure.com/143256\nReviewed-by: Brent Burgoyne <bburgoyne@instructure.com>\nTested-by: Jenkins\nProduct-Review: Ryan Shaw <ryan@instructure.com>\nQA-Review: Ryan Shaw <ryan@instructure.com>","distinct":false,"url":"https://api.github.com/repos/jfederico/canvas-lms/commits/ca30cb3035401f8142b9d428e35341eb21caf8f4"},{"sha":"9c4069b5c615b0170d49f717347d758f537c8c87","author":{"name":"Landon Gilbert-Bland","email":"f218d41c5d8cb90b84521a7fd658ba7f84020b6f@instructure.com"},"message":"Fix section specific selector showing up w/ FF disabled\n\nFixes COMMS-962\n\nTest Plan:\n  * Disable SSD feature flag\n  * Go to the create a discussion page\n  * Click on the `graded` discussion checkbox\n  * Make sure the section specific selector does not show up\n  * Make sure you can still create announcements\n\nChange-Id: I7f2e44f36d3bf44e84bf25e9fc76567f2c52838d\nReviewed-on: https://gerrit.instructure.com/143352\nReviewed-by: Venk Natarajan <vnatarajan@instructure.com>\nReviewed-by: Steven Burnett <sburnett@instructure.com>\nTested-by: Jenkins\nQA-Review: KC Naegle <knaegle@instructure.com>\nProduct-Review: KC Naegle <knaegle@instructure.com>","distinct":false,"url":"https://api.github.com/repos/jfederico/canvas-lms/commits/9c4069b5c615b0170d49f717347d758f537c8c87"},{"sha":"5c3283630304389e29bcf48adefb93b4e133b8c3","author":{"name":"Clay Diffrient","email":"169126af53a71f2221341d44650942640743f1c8@instructure.com"},"message":"Make the wiki page input box get focus properly\n\nThis adds a wait so that canvas-rce doesn't take\nfocus away from the input. There was already logic\nto do this, but some browsers (Firefox) regularly\nhandled it out of order making focus go to the RCE\ninstead.  Unfortunately, we need the `focus` option\nfor the RCE so that the sidebar registers the RCE\ninstance as the active RCE\n\ncloses CORE-1016\n\nTest Plan:\n  - Make a new wiki page\n  - Notice focus is on the title input\n\nChange-Id: I149a649a412302221a328c1bcf08eb11bf9e9e17\nReviewed-on: https://gerrit.instructure.com/142801\nReviewed-by: Ryan Shaw <ryan@instructure.com>\nTested-by: Jenkins\nQA-Review: Jeremy Putnam <jeremyp@instructure.com>\nProduct-Review: Clay Diffrient <cdiffrient@instructure.com>","distinct":false,"url":"https://api.github.com/repos/jfederico/canvas-lms/commits/5c3283630304389e29bcf48adefb93b4e133b8c3"},{"sha":"52a444543ce54b2627e1d7975314ebf55da38613","author":{"name":"Jeremy Neander","email":"f741c2bc464d3dd3dce932fc3824725070fe4f59@instructure.com"},"message":"spec: de-lint speed grader ruby spec\n\ntest plan:\n * verify Jenkins passes\n\nChange-Id: I2429690b23ac7bc68098a64f01776c1986d8a169\nReviewed-on: https://gerrit.instructure.com/143395\nTested-by: Jenkins\nReviewed-by: Keith T. Garner <kgarner@instructure.com>\nProduct-Review: Keith T. Garner <kgarner@instructure.com>\nQA-Review: Keith T. Garner <kgarner@instructure.com>","distinct":false,"url":"https://api.github.com/repos/jfederico/canvas-lms/commits/52a444543ce54b2627e1d7975314ebf55da38613"},{"sha":"58237bd8cf46f6dbd976ea9d0eb8607b78ae3d33","author":{"name":"Jon Willesen","email":"da3ef76ecad274f282c5b8d4f4fca01afef32e07@instructure.com"},"message":"fix planner override link header when cached\n\nWhen the response was cached, the Link header was not being populated\nbecause we never called API.paginate. This caused the planner to think\nthe first cached page it fetched was the only one.\n\nThe solution implemented here is the same one used by\nQuizzes::QuizzesApiController#index\n\nfixes ADMIN-848\n\ntest plan:\n- In the dashboard list view, have several pages of data and scroll\ninto the future and the past.\n- Refresh the page and do it again. You should be able to reach all the\nsame pages of data before the planner thinks it has loaded them all.\n\nChange-Id: I967a20a96cec623d5d9d2e1dc15f13aaab88006d\nReviewed-on: https://gerrit.instructure.com/143326\nReviewed-by: Mysti Sadler <mysti@instructure.com>\nTested-by: Jenkins\nQA-Review: Ed Schiebel <eschiebel@instructure.com>\nProduct-Review: Jon Willesen <jonw+gerrit@instructure.com>","distinct":false,"url":"https://api.github.com/repos/jfederico/canvas-lms/commits/58237bd8cf46f6dbd976ea9d0eb8607b78ae3d33"},{"sha":"7faa63a65e4d084a051ed18fdb497d7d6cb08b75","author":{"name":"Collin Parrish","email":"b9dadbb8afd4988124903f1fe8214a6fc5e094b2@instructure.com"},"message":"test instfs integration with submission discussions\n\ncloses RECNVS-343\n\ntest plan:\n    run this test in instfs e2e environment\n\nChange-Id: I00ebbc88adc91f2ab4a3364a11ebc2c2c603384b\nReviewed-on: https://gerrit.instructure.com/142292\nTested-by: Jenkins\nReviewed-by: Michael Jasper <mjasper@instructure.com>\nQA-Review: Collin Parrish <cparrish@instructure.com>\nProduct-Review: Collin Parrish <cparrish@instructure.com>","distinct":false,"url":"https://api.github.com/repos/jfederico/canvas-lms/commits/7faa63a65e4d084a051ed18fdb497d7d6cb08b75"},{"sha":"496a579b47066b778acaaba2f6f6a120cfe39b7d","author":{"name":"Jeremy Neander","email":"f741c2bc464d3dd3dce932fc3824725070fe4f59@instructure.com"},"message":"show assignment header menu button when name has focus\n\nfixes GRADE-912\n\ntest plan:\n 1. Visit New Gradebook\n 2. Set focus on the assignment link in an assignment header\n 3. Verify the options menu button is visible\n\nChange-Id: Ib1e19f6195695f428d5696747bf7ad2ae3aa2048\nReviewed-on: https://gerrit.instructure.com/143193\nTested-by: Jenkins\nReviewed-by: Spencer Olson <solson@instructure.com>\nReviewed-by: Derek Bender <djbender@instructure.com>\nQA-Review: Spencer Olson <solson@instructure.com>\nProduct-Review: Matt Goodwin <mattg@instructure.com>","distinct":false,"url":"https://api.github.com/repos/jfederico/canvas-lms/commits/496a579b47066b778acaaba2f6f6a120cfe39b7d"},{"sha":"a9a89c39936cb47c7bea356bcf7349096a02d4f8","author":{"name":"Robert Lamb","email":"dafb73202f40b34b36552bef79ffbf153345dff5@instructure.com"},"message":"spec: WIP - add pact provider files\n\nAdd pact provider files in Canvas base project following\nPact search conventions where pact_helper.rb is located in\ncanvas-lms/spec/service-consumer/\n\nAdd copyright to files\n\nproxy.app doesn't work yet - unable to create token\n\nChange-Id: I505a6165650ba1d89fcf054441e172e584a39305\nReviewed-on: https://gerrit.instructure.com/143296\nTested-by: Jenkins\nReviewed-by: Deepeeca Soundarrajan <dsoundarrajan@instructure.com>\nProduct-Review: Robert Lamb <rlamb@instructure.com>\nQA-Review: Robert Lamb <rlamb@instructure.com>","distinct":false,"url":"https://api.github.com/repos/jfederico/canvas-lms/commits/a9a89c39936cb47c7bea356bcf7349096a02d4f8"},{"sha":"51e8944c60e3ba42d07696d269470f9480ac6933","author":{"name":"Robert Lamb","email":"dafb73202f40b34b36552bef79ffbf153345dff5@instructure.com"},"message":"spec: fix flaky chrome specs for alerts\n\ntest plan: passes Jenkins\n\nadded waits_for_ajaximations\n\nChange-Id: I6af44abf2838feeeb0acc2de4959533d6133df0d\nReviewed-on: https://gerrit.instructure.com/143354\nTested-by: Jenkins\nReviewed-by: Deepeeca Soundarrajan <dsoundarrajan@instructure.com>\nProduct-Review: Robert Lamb <rlamb@instructure.com>\nQA-Review: Robert Lamb <rlamb@instructure.com>","distinct":false,"url":"https://api.github.com/repos/jfederico/canvas-lms/commits/51e8944c60e3ba42d07696d269470f9480ac6933"},{"sha":"1ede802a208dca2b5e5ccbd60ab5167d625fa0a9","author":{"name":"Adrian Packel","email":"3eea3249433fe2c0ab972c3e3285ffc37f044428@instructure.com"},"message":"Add course-level anonymous grading flag\n\nAdd a course-level development flag to enable AMM-specific anonymous\ngrading (separate from the existing \"anonymous grading\" flag), and\nensure it is only visible if AMM has been enabled on the relevant\naccount.\n\ncloses GRADE-920\n\nTest plan:\n* Have an account with a course attached to it.\n* Account-level testing:\n  * Open the feature options page for the account.\n  * Set Anonymous Moderated Marking on the account to \"OFF\", if it is\n    not already disabled.\n  * Reload the page.\n  * On the same page, in the course-level settings, you should NOT see\n    the \"Anonymous Grading (Anonymous Moderated Marking)\" flag (the one\n    added here).\n  * Set Anonymous Moderated Marking on the account to \"ON\" and reload.\n  * On the same page, in the course-level settings, you SHOULD see the\n    \"Anonymous Grading (Anonymous Moderated Marking)\" flag and be able\n    to set it to ALLOW or ON.\n* Course-level testing:\n  * Check that the feature option on the course level behaves\n    according to the value on the account level as usual (e.g.,\n    ON should force-enable it for the course while ALLOW should\n    make it optional).\n\n(Note that at present, if you disable the base Anonymous Moderated\nMarking feature at the account level, the new course-level setting will\nnot be adjusted accordingly and so could remain enabled even if AMM as a\nwhole is disabled. Since this is all still development-only and these\nflags are not intended to go live in their current form, this isn't a\nbig deal.)\n\nChange-Id: I84189ec6064403c7f109a85ca47f501c1e66b90c\nReviewed-on: https://gerrit.instructure.com/143103\nTested-by: Jenkins\nReviewed-by: Derek Bender <djbender@instructure.com>\nReviewed-by: Spencer Olson <solson@instructure.com>\nQA-Review: Shahbaz Javeed <sjaveed@instructure.com>\nProduct-Review: Sidharth Oberoi <soberoi@instructure.com>","distinct":false,"url":"https://api.github.com/repos/jfederico/canvas-lms/commits/1ede802a208dca2b5e5ccbd60ab5167d625fa0a9"},{"sha":"7ff5a1549a6cf363230c32a8ac36acaa86bf1c53","author":{"name":"Rob Orton","email":"250ddf768181782bd7ec5f89baf25f1862c7216e@instructure.com"},"message":"allow setting integration id on sections create\n\nfixes CORE-1139\n\ntest plan\n - create a section with integration_id\n - it should work\n\nChange-Id: If26a31ab3d3fcd442425a98c9fb74b32f7975f22\nReviewed-on: https://gerrit.instructure.com/143350\nTested-by: Jenkins\nReviewed-by: Cody Cutrer <cody@instructure.com>\nQA-Review: Jeremy Putnam <jeremyp@instructure.com>\nProduct-Review: Rob Orton <rob@instructure.com>","distinct":false,"url":"https://api.github.com/repos/jfederico/canvas-lms/commits/7ff5a1549a6cf363230c32a8ac36acaa86bf1c53"},{"sha":"a242a38e509ca316c3fe1d0dcf6d25ce87382517","author":{"name":"Mysti Sadler","email":"cc71b38622981b190802b4ee2a6d6a42da8392b8@instructure.com"},"message":"Order planner queries by overridden due dates\n\nfixes ADMIN-131\n\nTest plan\n- Set up user with several assignments with\n  as many types of overrides as possible\n- (User in multiple sections with different\n  due dates, user with an ad-hoc override,\n  overrides that don't override the due date, etc)\n- Ensure the API returns the items in the correct\n  order (assuming the user has a due date for the\n  item)\n- Check the same with ungraded quizzes\n- Ensure the plannable_date is the correct date\n  for the user\n- Turn off Canvas Planner and check to make sure\n  the to do list works correctly\n\nChange-Id: Ie38900ac645804867823ed24b59f228cefef7f51\nReviewed-on: https://gerrit.instructure.com/142412\nTested-by: Jenkins\nQA-Review: Deepeeca Soundarrajan <dsoundarrajan@instructure.com>\nReviewed-by: Jeremy Stanley <jeremy@instructure.com>\nProduct-Review: Mysti Sadler <mysti@instructure.com>","distinct":false,"url":"https://api.github.com/repos/jfederico/canvas-lms/commits/a242a38e509ca316c3fe1d0dcf6d25ce87382517"},{"sha":"79aa190c7871f3f6930a609895101c54b01c86ef","author":{"name":"Collin Parrish","email":"b9dadbb8afd4988124903f1fe8214a6fc5e094b2@instructure.com"},"message":"test instfs integration with quiz submissions\n\ncloses RECNVS-342\n\ntest plan:\n   test in instfs e2e environment\n\nChange-Id: Iadcb10f868d6503af5d3163da828b7a85db74f4f\nReviewed-on: https://gerrit.instructure.com/142188\nTested-by: Jenkins\nReviewed-by: Jacob Fugal <jacob@instructure.com>\nProduct-Review: Collin Parrish <cparrish@instructure.com>\nQA-Review: Collin Parrish <cparrish@instructure.com>","distinct":false,"url":"https://api.github.com/repos/jfederico/canvas-lms/commits/79aa190c7871f3f6930a609895101c54b01c86ef"},{"sha":"ee891dae4732cd7ce99e7c94baae9339de72b38f","author":{"name":"Jon Willesen","email":"da3ef76ecad274f282c5b8d4f4fca01afef32e07@instructure.com"},"message":"refactor new activity button's additional offset\n\nThis is a small, independant preparation step for the larger refactor.\n\ncloses ADMIN-850\n\ntest plan:\n- scrolling to new activity should still work as before.\n\nChange-Id: I65375977ae3924cb425edc33bb649354b4803052\nReviewed-on: https://gerrit.instructure.com/143267\nReviewed-by: Ed Schiebel <eschiebel@instructure.com>\nTested-by: Jenkins\nQA-Review: Deepeeca Soundarrajan <dsoundarrajan@instructure.com>\nProduct-Review: Jon Willesen <jonw+gerrit@instructure.com>","distinct":false,"url":"https://api.github.com/repos/jfederico/canvas-lms/commits/ee891dae4732cd7ce99e7c94baae9339de72b38f"},{"sha":"ec805f7e85b4d3ca18f1c7bc6228c85ab399e722","author":{"name":"Robert Lamb","email":"dafb73202f40b34b36552bef79ffbf153345dff5@instructure.com"},"message":"spec: fix flaky wiki-and-tiny spec\n\nuse <form element>.submit instead of clicking form button\n\nChange-Id: Ic5fba22f8fdf603fafff705ea18094a4ac30fd16\nReviewed-on: https://gerrit.instructure.com/143424\nTested-by: Jenkins\nReviewed-by: Jeremy Putnam <jeremyp@instructure.com>\nProduct-Review: Robert Lamb <rlamb@instructure.com>\nQA-Review: Robert Lamb <rlamb@instructure.com>","distinct":false,"url":"https://api.github.com/repos/jfederico/canvas-lms/commits/ec805f7e85b4d3ca18f1c7bc6228c85ab399e722"},{"sha":"2a1030288d08706d5d7e0340d24430db42364b42","author":{"name":"Matthew Berns","email":"ed5fb1ad870e0ed3c37350edfd73693a8642a101@instructure.com"},"message":"remove OutcomeActionsPopoverMenu and related specs\n\nwe stopped using the menu recently in\nhttps://gerrit.instructure.com/#/c/143168/\n\nleft code in place originally, but now just axing it since there\nseems to be intermittent spec failures with it, and it's sad to have\nbuilds failing due to an unused component\n\nChange-Id: I8390e5d916c1913dff880f1588ea8a499f2bfa6b\nReviewed-on: https://gerrit.instructure.com/143433\nTested-by: Jenkins\nReviewed-by: Michael Brewer-Davis <mbd@instructure.com>\nProduct-Review: Augusto Callejas <acallejas@instructure.com>\nQA-Review: Augusto Callejas <acallejas@instructure.com>","distinct":false,"url":"https://api.github.com/repos/jfederico/canvas-lms/commits/2a1030288d08706d5d7e0340d24430db42364b42"},{"sha":"917fc7a56880dff86c95e71393afb21e7cde9da4","author":{"name":"Jeremy Neander","email":"f741c2bc464d3dd3dce932fc3824725070fe4f59@instructure.com"},"message":"translate submission comments label\n\ncloses GRADE-952\n\ntest plan:\n 1. Change your locale to non-English\n 2. Visit New Gradebook\n 3. Open the Grade Detail Tray\n 4. Verify the \"Comments\" label is translated\n\nChange-Id: I7bc51e633c148cc794dc7445cf00580aee05b14d\nReviewed-on: https://gerrit.instructure.com/143518\nReviewed-by: Derek Bender <djbender@instructure.com>\nTested-by: Jenkins\nProduct-Review: Keith T. Garner <kgarner@instructure.com>\nQA-Review: Keith T. Garner <kgarner@instructure.com>","distinct":false,"url":"https://api.github.com/repos/jfederico/canvas-lms/commits/917fc7a56880dff86c95e71393afb21e7cde9da4"},{"sha":"259999f72f77fde2811c9d2b0dde77ec4aa92352","author":{"name":"Jeremy Neander","email":"f741c2bc464d3dd3dce932fc3824725070fe4f59@instructure.com"},"message":"spec: fix broken user spec\n\ntest plan:\n * verify Jenkins passes\n\nChange-Id: I05a9e8b16201d941a83f3ab52b325b09f6c7ffcf\nReviewed-on: https://gerrit.instructure.com/143510\nReviewed-by: Venk Natarajan <vnatarajan@instructure.com>\nTested-by: Jenkins\nProduct-Review: Jeremy Neander <jneander@instructure.com>\nQA-Review: Jeremy Neander <jneander@instructure.com>","distinct":false,"url":"https://api.github.com/repos/jfederico/canvas-lms/commits/259999f72f77fde2811c9d2b0dde77ec4aa92352"},{"sha":"068bdbce2636bcb575ee132140ad36132c8c3f58","author":{"name":"wdransfield","email":"0b22baa37a53e54134bd232c4a3511887bfd0a00@instructure.com"},"message":"Fix flakey group spec\n\nTest Plan:\nModified specs pass regardless of id array order\n\nChange-Id: I2f702a4878c6d64f9966d0cbf07ddbaca32a5705\nReviewed-on: https://gerrit.instructure.com/143476\nReviewed-by: Stewie aka Nicholas Stewart <nstewart@instructure.com>\nQA-Review: August Thornton <august@instructure.com>\nTested-by: Jenkins\nProduct-Review: Weston Dransfield <wdransfield@instructure.com>","distinct":false,"url":"https://api.github.com/repos/jfederico/canvas-lms/commits/068bdbce2636bcb575ee132140ad36132c8c3f58"}]}
{"actor":{"display_login":"jfederico"}}