-
Notifications
You must be signed in to change notification settings - Fork 479
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Ceara/aitt 543 565 more rubric amplitude #57786
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice progress Ceara! some comments inline.
For validation, I think it would make sense to verify that each event can make it into statsig, possibly with help from Nick. I believe statsig has support for local development, though you may have to ask Nick exactly how.
for testing on the ruby side, I think verifying the right calls are being made to Metrics::Events.log_event
would be sufficient. I believe this would be something like
Metrics::Events.stubs(:log_event).once
or possibly something like this if you want to verify params:
code-dot-org/dashboard/test/controllers/rubrics_controller_test.rb
Lines 51 to 53 in e474199
File.stubs(:write).with do |filename, contents| | |
filename == "#{Rails.root}/config/scripts_json/#{@lesson.script.name}.script_json" && contents.include?('ai-configured learning goal 1') | |
end.once |
const eventName = EVENTS.TA_RUBRIC_SECTION_AI_EVAL; | ||
analyticsReporter.sendEvent( | ||
eventName, | ||
{ | ||
...(reportingData || {}), | ||
rubricId: rubricId, | ||
sectionId: sectionId, | ||
}, | ||
PLATFORMS.BOTH | ||
); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
unfortunately I think what we really want here is to log an individual event for each student that is getting processed as a result of this request, which would need to be done on the backend. otherwise I'm worried it will be too hard to reconstruct within the statsig tool after the fact.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just to double check, you mean that we want an event for each student in this loop right before the evaluate rubric job is called?:
code-dot-org/dashboard/app/controllers/rubrics_controller.rb
Lines 196 to 200 in d5ed4e1
EvaluateRubricJob.perform_later( | |
user_id: @user.id, | |
requester_id: current_user.id, | |
script_level_id: script_level.id, | |
) |
Is this event in rubric settings still useful, or should it be removed? Does the event for the button click for an individual student need to be moved to the backend as well?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just to double check, you mean that we want an event for each student in this loop right before the evaluate rubric job is called?:
correct!
Is this event in rubric settings still useful, or should it be removed?
it could be useful, so since it is already implemented let's keep it.
Does the event for the button click for an individual student need to be moved to the backend as well?
let's keep this one in the frontend, since it could be useful to have this in amplitude too.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this event in rubric settings still useful, or should it be removed?
it could be useful, so since it is already implemented let's keep it.
if you keep it, it would be handy to have a different event name, so that we can easily distinguish between the teacher doing a thing and the individual student project being evaluated.
'levelId' => @level.id, | ||
'levelName' => @level.name, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this metadata will need to match up across event types, and will need to uniquely identify the level/lesson, keeping mind mind that levels are shared across ai-enabled units. I would recommend logging script name and level name.
on the client, script name and level name appear to already be available inside reportingData
.
I'm not sure exactly how this will work in statsig yet, but to minimize the chances of us running into problems, please use the exact same names for these fields across event types, e.g. "studentId" here if that's what we're using in other places.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated this
if ( | ||
experiments.isEnabled('ai-rubrics') && | ||
!!rubric && | ||
rubric.learningGoals.some(lg => lg.aiEnabled) | ||
) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this looks like a good start, but can you please further limit this so that we only log when the teacher is looking at the final level in the progression, where ai evaluation results are displayed. for example:
* should trigger on http://localhost-studio.code.org:3000/s/csd3-2023/lessons/11/levels/5?section_id=X&user_id=X
* should NOT trigger on http://localhost-studio.code.org:3000/s/csd3-2023/lessons/11/levels/4?section_id=X&user_id=X
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated this
metadata = { | ||
'studentId' => current_user.id, | ||
'unitName' => @script_level.script.name, | ||
'levelName' => @level.name, | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💯
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks great!
console.log(config); | ||
console.log(rubric); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove debug code?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
oof you're right
commit 2f2a3b704a12057cd887553e533ea3fbef9ae3eb Author: bencodeorg <ben@code.org> Date: Mon Apr 8 07:26:12 2024 -0700 Full width by default in simple dropdown (#57808) commit 98c6fa7aff8724aad45e34c75204d3c2e3b3ff7a Author: Liam Frye-Mason <liam.fryemason@code.org> Date: Mon Apr 8 06:58:20 2024 -0700 Add some more folders to eslint import rule (#57797) commit 5403a2239287dd0ec3b47af4845e5f60380b8207 Author: fisher-alice <107423305+fisher-alice@users.noreply.github.com> Date: Mon Apr 8 08:48:45 2024 -0500 Connect presentation view model card to redux (#57805) Connect presentation view model card to redux commit 3aff75ce4df0b5176bdd07622ccb81b6ae3bb1eb Merge: 934c8410d06 fbf61668344 Author: Brendan Reville <breville@users.noreply.github.com> Date: Mon Apr 8 22:29:04 2024 +1000 Merge pull request #57844 from code-dot-org/music-skip-progression Music: skip progression commit 934c8410d0640f230aceed8eff836c631d9d9f11 Merge: 6818dd353e8 6b0be1ad25d Author: Code.org Deploy Systems <dev@code.org> Date: Mon Apr 8 00:35:19 2024 -0700 Merge pull request #57855 from code-dot-org/dts_candidate_2024-04-08 DTS (Levelbuilder > Staging) [robo-dts] commit 6b0be1ad25d9c73fc66f72514b4d1286d30e3e11 Merge: 6818dd353e8 4743405eb41 Author: Continuous Integration <dev@code.org> Date: Mon Apr 8 07:35:12 2024 +0000 Merge remote-tracking branch 'origin/levelbuilder' into dts_candidate_2024-04-08 commit 6818dd353e80131abefa898cf9096cc2c3623068 Author: Continuous Integration <dev@code.org> Date: Mon Apr 8 07:30:17 2024 +0000 staging content changes (-robo-commit) commit 4743405eb4150733b02fa7e6f92c1fba52d929b9 Author: Continuous Integration <dev@code.org> Date: Mon Apr 8 07:30:16 2024 +0000 levelbuilder content changes (-robo-commit) commit 76d1f3c42a00fe93951b8840557c23e49517e476 Author: suresh <suresh@code.org> Date: Sat Apr 6 13:19:30 2024 -0700 Create & Update ParameterGroups in Preparation for Aurora 3 / MySQL 8 Upgrade (#57356) * Create new ParameterGroups in preparation of Aurora 3 upgrade. * Revert back to original Logical ID for ParameterGroup resources. * Specify Database Instance ParameterGroup for all instances on the Cluster Resource to support In Place upgrade to Aurora 3 / MySQL 8 * Preserve Description of existing ParameterGroups to prevent Replacement of the Resources. * Fixing copy/paste error. * Move all instance settings to the cluster. * Preserve the Aurora 2 Database Instance ParameterGroup. * Can't have a Resource with no Properties. * Fix copy/paste error with Description. * Ensure that existing clusters can be upgraded In Place from Aurora 2.x to 3.x. commit fbf61668344609e74e9a19f3069839567f0eebec Author: breville <brendan@code.org> Date: Fri Apr 5 21:17:00 2024 +0000 Panels support for skip button commit 6051a91aa58f48fd7d6e109fb1c6cc498d4903c6 Author: Dave Bailey <davidsbailey@users.noreply.github.com> Date: Fri Apr 5 13:41:47 2024 -0700 allow teacher to rerun ai eval when previous attempt resulted in error (#57836) * allow teacher to rerun ai eval when previous attempt resulted in error * revert debug code commit 72c8daee0086ae21cb8bb8a5f4fba8b763b4277e Author: breville <brendan@code.org> Date: Fri Apr 5 20:31:15 2024 +0000 Music: skip (WIP) commit b0e020b36fc6a219df81b394f3d49627c92a95be Author: Mike Harvey <43474485+mikeharv@users.noreply.github.com> Date: Fri Apr 5 16:30:45 2024 -0400 [Google Blockly] Field Angle Plugin (#57826) * check point: plugin class extended * simplify constructors - all params are optional * remove confusing comment commit 4f1a2c986ea2b916d218300876375901ec323e5c Merge: 824d1bc4e1b 3acedaff511 Author: Darin Webb <darin@code.org> Date: Fri Apr 5 13:27:01 2024 -0700 Merge pull request #57388 from code-dot-org/activejob-metrics-part-deux INF-1141 Publish additional ActiveJob metrics commit 824d1bc4e1b47b4c5c5fe3d3f2ae398da4bc9901 Author: Turner Riley <56283563+TurnerRiley@users.noreply.github.com> Date: Fri Apr 5 12:44:16 2024 -0700 NCES school data import for 2022 to 2023 (#57664) * setup district import * update school import * add comment * add one off script * use last years title i status * fix options spreading in csv read commit be7cf392612be9165fef89c7b50881a174371ae1 Author: Continuous Integration <dev@code.org> Date: Fri Apr 5 18:44:09 2024 +0000 Updated cookbook versions commit f01b5d2a0db82c886588eadd2b5b2e09ecc35b94 Author: Sanchit Malhotra <85528507+sanchitmalhotra126@users.noreply.github.com> Date: Fri Apr 5 11:27:42 2024 -0700 Implement existing effects in ToneJS (#57813) * Implement existing effects in ToneJS * Fix preview velocity * reliable key string commit 934e7721d31601eeab42e84b59f743f51f717894 Merge: 778ba7c0857 b6bde7529a2 Author: Darin Webb <darin@code.org> Date: Fri Apr 5 11:23:49 2024 -0700 Merge pull request #57834 from code-dot-org/darin/slow-cron slow activejob monitor cron commit 3acedaff511768dd868c2be068b28b021db4ff12 Author: Darin Webb <darin@code.org> Date: Fri Apr 5 12:58:47 2024 -0500 stub superclass logging in subclass tests commit b6bde7529a20316a396cfe871c1634d94e38e268 Author: Darin Webb <darin@code.org> Date: Fri Apr 5 10:43:46 2024 -0700 slow activejob monitor cron commit 778ba7c085731557e1851d4f0e8e7acd2e7a08ef Author: cearachew <ceara.chewning@code.org> Date: Fri Apr 5 13:19:44 2024 -0400 Small style change for the setting content width (#57831) commit 76d1465eb36b17365971619b769a5416e408242c Author: Kelby Hawn <9256643+kelbyhawn@users.noreply.github.com> Date: Fri Apr 5 09:55:42 2024 -0700 Add FAQ with video demo to code.org/ai-ta (#57830) commit df3228ca45280d04e1e8199adc2e9e6723ab3ea3 Merge: 689a495f5d2 92c171f1d80 Author: Eric Aderhold <eric.aderhold@code.org> Date: Fri Apr 5 09:27:34 2024 -0700 Merge pull request #57818 from code-dot-org/fix-csv-import Fix CSV reading for closed beta import commit 689a495f5d2949b855eea11177d164f76ee8cf5b Author: Jim Thomason <jim.thomason@code.org> Date: Fri Apr 5 11:22:24 2024 -0500 [JIMT] remove unused parts of Code Bridge (#57807) * jimt/cdo-ide-removal - removed the various components/functionality that we're not gonna use * jimt/cdo-ide-removal - tossed old config stuff commit 0ee339f3d84772a2444bfa2e25fa3eeb19eb9d18 Author: Artem Vavilov <artem.vavilov.7@gmail.com> Date: Fri Apr 5 19:08:59 2024 +0300 P20-788: Raise an error during user creation if LtiUserIdentity creation fails (#57806) commit 7774257896108d5498508758f6c603a3c9198672 Author: wilkie <david.wilkinson@code.org> Date: Fri Apr 5 12:01:08 2024 -0400 Fixes regression where the summary page would not render post AI eval. (#57828) commit 76988d579afefc5611270f4cb937e596032e16de Author: cearachew <ceara.chewning@code.org> Date: Fri Apr 5 11:24:13 2024 -0400 Ceara/aitt 543 565 more rubric amplitude (#57786) * Add student ID to the learning goal selection event * Add event names for individual and section ai run buttons * Add event to run student assessment button, pass reporting data through * Test for run ai for student event * Add amplitude event for running ai assessment for section * Add analytics event for loading the rubric page on a level w/ AI * Change analytics logging to use statsig AND amplitude * Add a statsig event when a student submits a project that is evaluated by ai * Add logging checks to ruby unit test * Change logged metadata for student submit ai eval event * Only log AI page visited if it is an assessment level * Add statsig logging for each student AI eval job from a section eval request * Also add section id to logging * Remove debug prints commit da01cbaad1a4eaa3fc8264d0b7d51cbaec417fdc Author: Continuous Integration <dev@code.org> Date: Fri Apr 5 14:29:33 2024 +0000 Update schema cache dump after schema changes. commit f50e0b9fe6afd67a641b550b71a27e2942ad56c4 Author: Kaitie O <kaitie@code.org> Date: Fri Apr 5 09:01:11 2024 -0500 changed color of assessment icon (#57693) * changed color * fix merge conflict * updated from staging commit e3c3710914f3d3b5c2670dfbd240fd24a8219d28 Merge: 2946056a21f d55f5142120 Author: Code.org Deploy Systems <dev@code.org> Date: Fri Apr 5 02:25:16 2024 -0700 Merge pull request #57825 from code-dot-org/dtl_candidate_297ac6d3 commit d55f51421200f69c6be19c18a499474e65b951e5 Merge: 2946056a21f 297ac6d361b Author: Continuous Integration <dev@code.org> Date: Fri Apr 5 09:20:09 2024 +0000 Merge commit '297ac6d3' into dtl_candidate_297ac6d3 commit 9993b8949872d93f0431c3dcf843e85a6d8ab4de Merge: a6a91af1751 8f7dae492e9 Author: Code.org Deploy Systems <dev@code.org> Date: Fri Apr 5 00:35:24 2024 -0700 Merge pull request #57824 from code-dot-org/dts_candidate_2024-04-05 DTS (Levelbuilder > Staging) [robo-dts] commit 8f7dae492e913eec135d9f1e785506faf9a896cb Merge: a6a91af1751 2946056a21f Author: Continuous Integration <dev@code.org> Date: Fri Apr 5 07:35:16 2024 +0000 Merge remote-tracking branch 'origin/levelbuilder' into dts_candidate_2024-04-05 commit 2946056a21fd94f11f0cc24b170e502c327a089f Author: Continuous Integration <dev@code.org> Date: Fri Apr 5 07:30:25 2024 +0000 levelbuilder content changes (-robo-commit) commit a6a91af1751568ad3a38e72eaa69311468505a3e Author: Continuous Integration <dev@code.org> Date: Fri Apr 5 07:30:22 2024 +0000 staging content changes (-robo-commit) commit 297ac6d361b06cdcd92ece921f077887b17d9481 Merge: 041aee6c62f 87a0d47bcdb Author: Code.org Deploy Systems <dev@code.org> Date: Thu Apr 4 18:12:17 2024 -0700 Merge pull request #57822 from code-dot-org/staging DTT (Staging > Test) [robo-dtt] commit 87a0d47bcdb0bfdd5db2c7fc42f389a064682191 Author: Mike Harvey <43474485+mikeharv@users.noreply.github.com> Date: Thu Apr 4 21:11:38 2024 -0400 [Google Blockly] xmlToCode (#57803) commit 041aee6c62fbd99e6d3aa3fdcd92609adde80c5a Merge: daff873a088 4cd68cbfb6d Author: Code.org Deploy Systems <dev@code.org> Date: Thu Apr 4 16:58:12 2024 -0700 Merge pull request #57820 from code-dot-org/staging DTT (Staging > Test) [robo-dtt] commit 4cd68cbfb6da8f96847fc44db2b2a9b526bbdc54 Author: Elijah Hamovitz <elijahhamovitz@gmail.com> Date: Thu Apr 4 16:16:06 2024 -0700 Fix Violations of and Reenable `Style/ExplicitBlockArgument` (#57745) * Fix Violations of and Reenable `Style/ExplicitBlockArgument` > Enforces the use of explicit block argument to avoid writing block literal that just passes its arguments to another block. I don't have a strong feeling about this one; I generally agree that explicit is better than implicit, but on the other hand I think recognizing what `&block` is doing takes more Ruby-specific knowledge than `yield`. I'm inclined to say `Style/` Cops should be enabled unless we have a good reason not to, so I'm open to hearing good reasons if anyone has one! - https://www.rubydoc.info/gems/rubocop/RuboCop/Cop/Style/ExplicitBlockArgument - https://docs.rubocop.org/rubocop/cops_style.html#styleexplicitblockargument * manually add method argument that Rubocop's autocorrect somehow missed commit 92c171f1d80c621748584f6afbaedec7dff7f84b Author: Eric Aderhold <eric.aderhold@code.org> Date: Thu Apr 4 16:14:03 2024 -0700 Fix CSV reading for closed beta import commit f8438f8225eaf83147c8f310c543117583b06682 Author: juanmanzojr <137838584+juanmanzojr@users.noreply.github.com> Date: Thu Apr 4 15:51:43 2024 -0700 Revert "Edit student information modal conditions for retrieving U13 US State…" (#57816) This reverts commit d3d4e5d614f2748c8b2dfb5843896a145607c74e. commit b3b8497fbf97ef63e287f746a607fea643ec297f Author: Nick Lathe <nick.lathe@code.org> Date: Thu Apr 4 15:34:41 2024 -0700 Add Schoology section to LTI onboarding docs (#57696) * Add Schoology section to LTI onboarding docs * Update doc with new portal link * Update Canvas section Signed-off-by: Nick Lathe <nick.lathe@code.org> commit d3d4e5d614f2748c8b2dfb5843896a145607c74e Author: juanmanzojr <137838584+juanmanzojr@users.noreply.github.com> Date: Thu Apr 4 15:10:03 2024 -0700 Edit student information modal conditions for retrieving U13 US State information (CPA) (#57785) * Modal for CPA u13 state retrieval * WIP * Testing for cpa conditions in modal * Add condition for lti student * Reload current page when submitting state modal * Create conditon variables for readability commit 80b47dd67b9dda781e7fcbd53ddd1c64334daaed Author: juanmanzojr <137838584+juanmanzojr@users.noreply.github.com> Date: Thu Apr 4 14:50:23 2024 -0700 Supported placements URL for LTI launch outside of context (#57800) commit daff873a088ae9df30e9a225498c5c57b342e3da Merge: 6b1705c3629 f8ded42001d Author: Code.org Deploy Systems <dev@code.org> Date: Thu Apr 4 13:38:10 2024 -0700 Merge pull request #57810 from code-dot-org/staging DTT (Staging > Test) [robo-dtt] commit f8ded42001df2909d70e43ecd6486fbd04db27a1 Author: Mike Harvey <43474485+mikeharv@users.noreply.github.com> Date: Thu Apr 4 16:37:44 2024 -0400 Hide deprecated projects from personal project table (#57776) * Hide deprecated projects from personal project table * remove unused import commit 8d97a03cfa4c83e1012bd7772e573805c7063ca5 Author: bencodeorg <ben@code.org> Date: Thu Apr 4 12:44:20 2024 -0700 Gen AI: initial "compare models" dialog (#57775) * Use new design system Button component * Move presentation toggle into container * Use new Button in submit area * Remove inline styles * Checkpoint: choose model dropdown and button * Scrollable body * View more button when content is too long * Only use one styling file * Break component into its own file * Remove one last inline style * Reset shared dropdown styling * Remove unnecessary div, break up render function * Update text, use more specific selector commit 8a6789c23b50040a94e576ed68cc273f91d99726 Author: Kelby Hawn <9256643+kelbyhawn@users.noreply.github.com> Date: Thu Apr 4 12:39:24 2024 -0700 LinkButton component style updates (#57764) * add a tag selectors on _baseButton.module.scss * add border to all buttons and misc enhancements * update /button/CHANGELOG.md * update changelog, fixed active buttons being jumpy, set default border color to be transparent --------- Co-authored-by: denyslevada <levada.denys@gmail.com> commit fbd44aac9907f4d3b84e8ca4ff0f25ac4bee90ba Merge: 33c1163348c b72a77895df Author: Darin Webb <darin@code.org> Date: Thu Apr 4 14:13:03 2024 -0500 Merge branch 'staging' of github.com:code-dot-org/code-dot-org into activejob-metrics-part-deux commit 33c1163348ce0f7ce6e82389c19cff747ee7591d Author: Darin Webb <darin@code.org> Date: Thu Apr 4 14:12:44 2024 -0500 cleanup whitespace commit b72a77895dfe6cd5a5c96a8b850d87f9c53394f9 Author: Mike Harvey <43474485+mikeharv@users.noreply.github.com> Date: Thu Apr 4 15:04:27 2024 -0400 [Google Blockly] Move properties to list in constants (#57796) * [Google Blockly] Move properties to list in constants * Update constants.ts commit 01af5a88805024d3996bcd2f3683064ebf55004c Author: Turner Riley <56283563+TurnerRiley@users.noreply.github.com> Date: Thu Apr 4 11:06:23 2024 -0700 address feedback (#57799) commit 6b1705c3629c09e2b06897d74ad024c8bcf32577 Merge: e549029139d fe6b9c22180 Author: Code.org Deploy Systems <dev@code.org> Date: Thu Apr 4 10:52:10 2024 -0700 Merge pull request #57801 from code-dot-org/staging DTT (Staging > Test) [robo-dtt] commit fe6b9c22180300685a4f306b9103279496f63ff8 Author: Hannah Bergam <hannahbergam@gmail.com> Date: Thu Apr 4 10:48:04 2024 -0700 Zip Search Populating School Dropdown List (#57604) * zip search dropdown populating * fixing continuous api calls * made it so switching back to dropdown updates back to selectASchool option * pr feedback * test is tantalizingly close to passing * test is passing * one last bit of pr feedback commit 1bcc596b5ca59fea56671029017bb2ff561f5838 Author: wilkie <david.wilkinson@code.org> Date: Thu Apr 4 13:42:58 2024 -0400 Aitt evidence goto line (#57641) * Adds abstracted ability to 'goto' the line of the current editor. Uses that to scroll to the evidence pointed to by the AI in the AI TA. * Updates rendering logic to presume annotations from observations are already parsed out. * Adds a clear table describing the fallback situations. * Adds tests for the annotator and the existence and clicking of links. * Fixes lint errors. commit 7b0c70029a99cc29ecb0a1d495b712865722f116 Author: Mike Harvey <43474485+mikeharv@users.noreply.github.com> Date: Thu Apr 4 13:26:46 2024 -0400 [Google Blockly] Image and Label fields for K1 blocks (#57778) * [Google Blockly] Image and Label fields for K1 blocks * fix unit test commit 8d0873d58096d98dd6ade931272db56fd2d69f80 Author: Jim Thomason <jim.thomason@code.org> Date: Thu Apr 4 12:09:59 2024 -0500 [JIMT][CDOIDE] (#57419) * jimt/cdo-ide-import - check pointing myself with a mostly working one * jimt/cdo-ide-import - fixed ts error * jimt/cdo-ide-import - fixed json editor crash * jimt/cdo-ide-import - added sample json file * jimt/cdo-ide-import - added some additional typing * jimt/cdo-ide-import - can no longer preview json or javascript files * jimt/cdo-ide-import - shuffled deck chairs * jimt/cdo-ide-import - finished type twiddling * jimt/cdo-ide-import - twiddled package.json * jimt/cdo-ide-import - temporarily commented out JSPreview, since websandbox is no longer installed * jimt/cdo-ide-import - put back missing props * jimt/cdo-ide-import - put back missing props commit b8749d93fb0ef1c3d7000e367e3fc899b71e69dd Author: Sanchit Malhotra <85528507+sanchitmalhotra126@users.noreply.github.com> Date: Thu Apr 4 12:51:19 2024 -0400 Rework AI Chat types (part 2) (#57744) * Rename initial ai customizations -> aichat settings * update types * fix name * fix name * make hidePresentationPanel required commit 03138a015f0e1a91a6aa5892a0512e0ccc00f570 Author: Kelby Hawn <9256643+kelbyhawn@users.noreply.github.com> Date: Thu Apr 4 09:47:13 2024 -0700 Replace Projects with Districts link in signed out top nav on code.org (#56941) * update hamburger.rb with districts string * update test_hamburger.rb * update header.feature commit e549029139d52910ca7a2f6539b1b9e9c9505fea Merge: cc2116ec338 1cf11dfe2bf Author: Code.org Deploy Systems <dev@code.org> Date: Thu Apr 4 09:36:18 2024 -0700 Merge pull request #57798 from code-dot-org/staging DTT (Staging > Test) [robo-dtt] commit 1cf11dfe2bfc2b0cf114837ceac7f17d61b62c45 Author: Turner Riley <56283563+TurnerRiley@users.noreply.github.com> Date: Thu Apr 4 09:33:10 2024 -0700 Update Curriculum Catalog UI tests to check for Stretch Recommended Curriculum (#57514) * set up stretch framework * add test * update test comments * update expanded card to show stretch curriculum * remove console logs * get next stretch curriculum if first result is the same as the similar curriculum * add tests * update ui test * simplify and fix taught curriculum test * update recommender tests to include new test curriculum commit 9c8b5b6d441579335e68f0e9844d3f0a18c0d23a Author: Turner Riley <56283563+TurnerRiley@users.noreply.github.com> Date: Thu Apr 4 09:28:13 2024 -0700 Cache Stretch Curriculum Recommender results (#57522) * set up stretch framework * add test * update test comments * update expanded card to show stretch curriculum * remove console logs * get next stretch curriculum if first result is the same as the similar curriculum * add tests * add caching for stretch curriculum * simplify and fix taught curriculum test * update recommender tests to include new test curriculum commit 62a3f4c1eca31ed539218f0773118834bf5b08d1 Author: Liam Frye-Mason <liam.fryemason@code.org> Date: Thu Apr 4 09:13:42 2024 -0700 [ProgressV2] Making small accessibility changes (#57784) * [ProgressV2] Making small accessibility changes * Add label * fix commit 5a40c8356dd5ed9208821397013a06cc6701193c Merge: fb5efcd0a2e 6bed41b9509 Author: Code.org Deploy Systems <dev@code.org> Date: Thu Apr 4 02:25:21 2024 -0700 Merge pull request #57795 from code-dot-org/dtl_candidate_07cc60e9 commit 6bed41b9509b600b7290888836c7991680a3ae29 Merge: fb5efcd0a2e 07cc60e9609 Author: Continuous Integration <dev@code.org> Date: Thu Apr 4 09:20:13 2024 +0000 Merge commit '07cc60e9' into dtl_candidate_07cc60e9 commit d5ed4e18f86377b9d0ad83fa12ae94d60784dd96 Merge: b5637ff4e46 bd36643ea38 Author: Code.org Deploy Systems <dev@code.org> Date: Thu Apr 4 00:35:20 2024 -0700 Merge pull request #57793 from code-dot-org/dts_candidate_2024-04-04 DTS (Levelbuilder > Staging) [robo-dts] commit bd36643ea38203f1bcff009bacd4f790e8b0d1a4 Merge: b5637ff4e46 fb5efcd0a2e Author: Continuous Integration <dev@code.org> Date: Thu Apr 4 07:35:12 2024 +0000 Merge remote-tracking branch 'origin/levelbuilder' into dts_candidate_2024-04-04 commit fb5efcd0a2edbac9c0978b6dac992502691eb090 Author: Continuous Integration <dev@code.org> Date: Thu Apr 4 07:30:27 2024 +0000 levelbuilder content changes (-robo-commit) commit b5637ff4e4679d4f546f8e54486d81f0c6ebc5c2 Author: Continuous Integration <dev@code.org> Date: Thu Apr 4 07:30:09 2024 +0000 staging content changes (-robo-commit) commit c99213e451b7e2ea6a22925a4a725853b468299a Author: wilkie <david.wilkinson@code.org> Date: Wed Apr 3 22:59:20 2024 -0400 Adds back the rubric header padding on the left/right buttons. (#57791) commit cc2116ec338f5d90f145d8ae5864426136fc3e97 Merge: 07cc60e9609 8130de45a55 Author: Code.org Deploy Systems <dev@code.org> Date: Wed Apr 3 19:58:11 2024 -0700 Merge pull request #57792 from code-dot-org/staging DTT (Staging > Test) [robo-dtt] commit 8130de45a554e7cc60f832d8845c16584a7f5410 Author: Dave Bailey <davidsbailey@users.noreply.github.com> Date: Wed Apr 3 19:54:02 2024 -0700 enable ai evaluation in other copies of csd3 (#57762) * add other csd3 clones to ruby ai config * mark learning goals as ai-enabled via levelbuilder ui * make newly ai-enabled units available in ui tests commit 07cc60e9609bc1c9f37d0afca404e3ab43fcb1df Merge: 34d959c7c20 62840116750 Author: Code.org Deploy Systems <dev@code.org> Date: Wed Apr 3 18:16:10 2024 -0700 Merge pull request #57789 from code-dot-org/staging DTT (Staging > Test) [robo-dtt] commit 62840116750c0aa21e28a8754ed1fc3ba1d18b83 Merge: dfa456645c3 511f119957b Author: Brendan Reville <breville@users.noreply.github.com> Date: Thu Apr 4 12:11:00 2024 +1100 Merge pull request #57783 from code-dot-org/music-one-pack-fix Music: fix library filtering commit dfa456645c3bf1c6b74404b81211c7ab33c3d029 Merge: 0293a9799cd 4ac2d828e53 Author: Seth Nickell <seth.nickell@code.org> Date: Wed Apr 3 18:08:22 2024 -0700 Merge pull request #57772 from code-dot-org/P20-829/prepare-units-on-demand P20-829: Undo optimization of deprecated unit fixtures seeding commit 0293a9799cddc3ff178da39f0c8cb7c6dd4cc01f Author: Turner Riley <56283563+TurnerRiley@users.noreply.github.com> Date: Wed Apr 3 17:32:11 2024 -0700 update license and notice (#57779) commit e5a7215b6168400d8375a8ef4a76f7cdd0a2b525 Author: Hannah Bergam <hannahbergam@gmail.com> Date: Wed Apr 3 16:36:06 2024 -0700 Adds event logging to existing school association flow (#57734) * adding constants and two event logs * relabeling userType to be more explicit, updating experiment param * added manual checked logging, fixed school selected event accordingly commit 655979e71e98f2ebf894d5bb8116237e8139a83a Merge: 3366bdf5645 91995af8beb Author: Mario Gil Correa <66776217+mgc1194@users.noreply.github.com> Date: Wed Apr 3 16:19:59 2024 -0700 Merge pull request #57749 from code-dot-org/i18n-sync-down-out-04-03-2024 I18n sync Down & Out 04/03 commit 511f119957b7d71239c599009df19ec02dde75d1 Author: breville <brendan@code.org> Date: Wed Apr 3 22:50:01 2024 +0000 Working fix commit 3366bdf5645d7d690180d5d382d9d16ddfd6dee7 Author: Turner Riley <56283563+TurnerRiley@users.noreply.github.com> Date: Wed Apr 3 15:46:59 2024 -0700 Implement Stretch Curriculum Recommender on the Curriculum Catalog page (#57460) * set up stretch framework * add test * update test comments * update expanded card to show stretch curriculum * remove console logs * get next stretch curriculum if first result is the same as the similar curriculum * add tests * simplify and fix taught curriculum test * update recommender tests to include new test curriculum * ensure similar and stretch overlap to test that behavior * remove duplicate test * remove unnecessary style lines commit e3dba1bcebce46e7622a1c61ca04036e3c77381b Author: breville <brendan@code.org> Date: Wed Apr 3 22:22:16 2024 +0000 Music: fix library filtering commit fc19b5ce914e7e71e4252a49e00d5af21d21c6a1 Author: Turner Riley <56283563+TurnerRiley@users.noreply.github.com> Date: Wed Apr 3 15:06:44 2024 -0700 Update Terms of Service page (#57739) * update tos page * pr feedback * remove use of screenname commit b1f885711e021a15dc8b44bec0ae7dd333128ef0 Author: levadadenys <levada.denys@gmail.com> Date: Thu Apr 4 00:36:41 2024 +0300 [DESIGN2-95] Tabs component (#57195) * Created Tabs component * created tests and stories for Tabs component * created documentation for Tabs component commit 878d262979cf0ea395c7679026ef7b8dd851b6ce Author: Continuous Integration <dev@code.org> Date: Wed Apr 3 21:36:31 2024 +0000 levelbuilder content changes (-Vijaya) commit 3988f9771955447bc60ecf62787c9339b9a7faba Author: Mike Harvey <43474485+mikeharv@users.noreply.github.com> Date: Wed Apr 3 17:11:40 2024 -0400 [Google Blockly] support gray undeletable blocks (#57755) * [Google Blockly] support gray undeletable blocks * check read-only workspace commit 04341e491657cd198fe493fae34adf1579a2b327 Author: Continuous Integration <dev@code.org> Date: Wed Apr 3 21:07:11 2024 +0000 levelbuilder content changes (-Vijaya) commit 951ca524b38789fa580081ac3c2ece52fe4f2e00 Author: Continuous Integration <dev@code.org> Date: Wed Apr 3 21:04:52 2024 +0000 levelbuilder content changes (-Vijaya) commit b62be6ddd6c66b1f7391b8e5bfd46e12ece067d3 Author: Turner Riley <56283563+TurnerRiley@users.noreply.github.com> Date: Wed Apr 3 14:03:57 2024 -0700 Update /privacy page (#57742) * update privacy page * remove double spaces * ap csa commit ffd3fdf3d33f7cbfaae2e411703645ee1488adba Merge: dcfdf952517 30c462fec87 Author: Jessica Kulwik <jessica@code.org> Date: Wed Apr 3 13:55:26 2024 -0700 Merge pull request #57519 from code-dot-org/jmkulwik/csp-create-task Update the PR template to warn about the CSP Create task. commit dcfdf9525172869d538b251713a3ae63d073c62b Author: fisher-alice <107423305+fisher-alice@users.noreply.github.com> Date: Wed Apr 3 15:48:52 2024 -0500 Add Gen AI Presentation view of model card UI (#57716) Add Gen AI Presentation view of model card UI commit f2bcd8763acd472bcfda2c20a6d8ff7587add50b Author: Vijaya Manohararaj <124813947+vijayamanohararaj@users.noreply.github.com> Date: Wed Apr 3 13:40:23 2024 -0700 Script to generate commands for cloning curriculum scripts. (#57663) commit b72436fcf3b1e72a71afaef923c16d46e6f5b716 Merge: ebb6d07565d e6dfee46efa Author: Seth Nickell <seth.nickell@code.org> Date: Wed Apr 3 13:20:49 2024 -0700 Merge pull request #57735 from code-dot-org/datablock/no-covid-countries Don't try to run broken update for 'COVID-19 Cases per Country' commit ebb6d07565d395f5553addc2a971ca6538cf46f5 Merge: 07bb75b4071 1c35998f578 Author: Brendan Reville <breville@users.noreply.github.com> Date: Thu Apr 4 07:19:06 2024 +1100 Merge pull request #57750 from code-dot-org/panels-two-more-corners commit 07bb75b4071e8dcac2b230532b37aa5838ef261c Author: Nick Lathe <nick.lathe@code.org> Date: Wed Apr 3 12:55:41 2024 -0700 Nicklathe/debug statsig (#57759) - Init Statsig in rails initializer for all environments, not just development. This ensures it's running and available in all worker threads Signed-off-by: Nick Lathe <nick.lathe@code.org> commit 83ac86b0967eb77f0dd4a8726d58f661ef48d6fa Author: wilkie <david.wilkinson@code.org> Date: Wed Apr 3 15:37:13 2024 -0400 Fixes issues with malformed evidence strings. (#57489) * Fixes issues with malformed evidence strings. Since evidence is its own column, we can get away with not having some of the information and falling back to what is there. If the code isn't there, we still annotate with the message for those lines. If the message isn't there, then we annotate with the observations column instead. Here, we only use the line numbers to mark the highlights. The evidence listing is just a bulleted list of sentences provided in the observations column. We also allow for partial matches of code. The AI sometimes truncates the code down (e.g. 'if (something) { ... }') and in those cases we want to gracefully fall back to highlighting the parts we match. It's not ideal. * Adds better documentation for the changes to annotateLines. * Encapsulates stubs in LearningGoalsTest to limit their scope. * Cleans up lack of safe navigation. commit 17973fc3c673ef16c75333ab38f8de89a3f07ff8 Author: wilkie <david.wilkinson@code.org> Date: Wed Apr 3 15:36:04 2024 -0400 Removes outline for active learning goal rubric left/right button. (#57722) commit 34d959c7c20aceb683d4dbf92a02eb946e8ee5fc Merge: aee97598433 9616974f6cb Author: Code.org Deploy Systems <dev@code.org> Date: Wed Apr 3 12:26:11 2024 -0700 Merge pull request #57773 from code-dot-org/staging DTT (Staging > Test) [robo-dtt] commit 9616974f6cb7f2146bdd5c6ab1729f7df7356117 Author: Kelby Hawn <9256643+kelbyhawn@users.noreply.github.com> Date: Wed Apr 3 11:52:36 2024 -0700 Add ariaLabel prop to LinkButton on ActionBlocksWrapper component (#57767) * add ariaLabel prop to ActionBlocksWrapper.jsx * add ariaLabel to primary buttons on ActionBlocksWrapper.story.jsx commit eeb51bed9d847b4d8675ae68410c1e03083d562b Author: Liam Frye-Mason <liam.fryemason@code.org> Date: Wed Apr 3 11:45:18 2024 -0700 [ProgressV2] Make progress table more screen reader accessible (#57718) * [ProgressV2] Make progress table more screen reader accessible * Functionize * Fix * Rename * Add caption * Fix * Add header * Fix tests * Fix tests again * Fix the most annoying test ever * Fix more tests commit fa6c321000fc37f6aea421687fae5bfc25bcf8f2 Author: Vijaya Manohararaj <124813947+vijayamanohararaj@users.noreply.github.com> Date: Wed Apr 3 11:39:13 2024 -0700 RTL Migration for two tests in UnitEditorTests for levelbuilder UnitEditor page. (#57654) RTL Migration for two tests in UnitEditorTests for levelbuilder. commit 4ac2d828e5373cac3bf3bf070df85a97c7512cb0 Author: Artem Vavilov <artem.vavilov.7@gmail.com> Date: Wed Apr 3 21:36:22 2024 +0300 P20-829: Undo the deprecated unit fixtures seeding optimization commit aee975984339fab1db674c9af7abfaa83ba40c22 Merge: abccbb45853 946957850cc Author: Seth Nickell <snickell@gmail.com> Date: Wed Apr 3 11:29:53 2024 -0700 Merge pull request #57771 from code-dot-org/staging DTT (Staging > Test) [seth] commit 946957850cc99544bef96afb6ead33277a8d21a3 Merge: 5c8c4062c94 9b2cdb48845 Author: Seth Nickell <seth.nickell@code.org> Date: Wed Apr 3 11:28:47 2024 -0700 Merge pull request #57770 from code-dot-org/P20-829/prepare-units-on-demand P20-829: Fix `DBQueryTest` commit 9b2cdb48845a12c6aa099da0139b327a86f4ac63 Author: Artem Vavilov <artem.vavilov.7@gmail.com> Date: Wed Apr 3 21:25:26 2024 +0300 P20-829: Fix `DBQueryTest` commit 5c8c4062c94cbefc5153bd6375741694c4cfccfb Author: Elijah Hamovitz <elijahhamovitz@gmail.com> Date: Wed Apr 3 11:24:03 2024 -0700 Fix Violations of and Reenable `Style/AccessModifierDeclarations` (#57621) * Fix Violations of and Reenable `Style/AccessModifierDeclarations` > Access modifiers should be declared to apply to a group of methods or inline before each method, depending on configuration. EnforcedStyle config covers only method definitions. Applications of visibility methods to symbols can be controlled using AllowModifiersOnSymbols config. This is sort of a weird one. There are two ways to declare methods as `private` in Ruby; either in a group like: ``` class Foo private def bar; end def baz; end end ``` or inline like: ``` class Foo private def bar; end private def baz; end end ``` This Cop attempts to enforce consistency; in our case, consistently using the inline rather than the group approach. I personally prefer inline; it's slightly more verbose, but eliminates the potential confusion of looking at methods declared at the bottom of a file and not realizing that they're all private because the single declaration of that is offscreen. What do folks think? Would anyone prefer to enforce the group style, or not enforce anything at all? Fix applied automatically with `bundle exec rubocop --autocorrect-all --only Style/AccessModifierDeclarations`. I then automatically fixed up a few resulting excess newlines with `bundle exec rubocop --autocorrect-all --only Layout/EmptyLines` - https://docs.rubocop.org/rubocop/cops_style.html#styleaccessmodifierdeclarations - https://www.rubydoc.info/gems/rubocop/RuboCop/Cop/Style/AccessModifierDeclarations * remove change that was intended for a different branch * autocorrect a few new violations of Style/AccessModifierDeclarations since this PR was created commit 8dbc26aeec66d838ff27b8594d396bca126a8598 Author: Liam Frye-Mason <liam.fryemason@code.org> Date: Wed Apr 3 11:16:54 2024 -0700 [ProgressV2] Add accessible icon labels (#57728) * [ProgressV2] Add accessible icon labels * fix * Fix test * Fix tests * Fix test * Fix tests * Change label text commit 70457148de01aa91ee06861dc036c3da357a5f2e Merge: a9c6dbb1a49 e4d2b6715a3 Author: Dave Bailey <davidsbailey@users.noreply.github.com> Date: Wed Apr 3 11:14:32 2024 -0700 Merge pull request #57769 from code-dot-org/merge-lb-to-staging-again manual DTS (Levelbuilder > Staging) commit e4d2b6715a3ce29bee5bf28ad7faf661131b5e21 Merge: 79e8cc8368b a9c6dbb1a49 Author: David Bailey <davidsbailey@users.noreply.github.com> Date: Wed Apr 3 11:11:25 2024 -0700 Merge remote-tracking branch 'origin/staging' into merge-lb-to-staging-again commit 79e8cc8368ba75e7edf31d018db10346954cfea5 Author: Continuous Integration <dev@code.org> Date: Wed Apr 3 18:07:37 2024 +0000 levelbuilder content changes (-dave) commit abccbb458530df5d101449a4c993d7ff6a9159c4 Merge: 268df939107 a9c6dbb1a49 Author: Seth Nickell <snickell@gmail.com> Date: Wed Apr 3 10:38:42 2024 -0700 Merge pull request #57766 from code-dot-org/staging DTT (Staging > Test) [seth] commit a9c6dbb1a491ee5217be6e21719f3db5da172cad Merge: 52a0ae178ba 8a5cf07dee6 Author: Seth Nickell <seth.nickell@code.org> Date: Wed Apr 3 10:32:22 2024 -0700 Merge pull request #57765 from code-dot-org/P20-829/prepare-units-on-demand P20-829: Fix `ScriptsControllerTest` and `ScriptLevelsHelperTest` commit 52a0ae178ba9e1a57ffc6f65f7fc5202e396285c Author: Sanchit Malhotra <85528507+sanchitmalhotra126@users.noreply.github.com> Date: Wed Apr 3 13:25:07 2024 -0400 Music: fix bug loading initial level data (#57747) commit 8a5cf07dee6a69762fb14b6e0ffc6bc953218ef0 Author: Artem Vavilov <artem.vavilov.7@gmail.com> Date: Wed Apr 3 20:17:41 2024 +0300 P20-829: Fix `ScriptsControllerTest` and `ScriptLevelsHelperTest` commit 3aff4bcbb32a904d1e8ad24f9c3852cc03161a38 Author: Continuous Integration <dev@code.org> Date: Wed Apr 3 17:15:03 2024 +0000 levelbuilder content changes (-seth) commit 07ec8d3fd6d60ec86e505b1c7618aadb865a2d5f Merge: e069573fbde 21c14b9c5c9 Author: Continuous Integration <dev@code.org> Date: Wed Apr 3 17:12:41 2024 +0000 Merge branch 'staging' of github.com:code-dot-org/code-dot-org into staging commit 21c14b9c5c9c2edc65111afb229ab773fab58670 Merge: 3a278852ffc a7fd2578c55 Author: Stephen Liang <stephenliang@users.noreply.github.com> Date: Wed Apr 3 10:10:24 2024 -0700 Merge pull request #57423 from code-dot-org/stephen/eyes-flaky-header fix(eyes): wait for fade animation to complete before snapshotting commit e069573fbdeff96644618f00093bb57664c65bb3 Author: Continuous Integration <dev@code.org> Date: Wed Apr 3 17:02:16 2024 +0000 update trusted proxy list for cloudfront servers commit 3a278852ffc005cf4e346f619795522ee088aecd Author: bencodeorg <ben@code.org> Date: Wed Apr 3 09:56:48 2024 -0700 Gen AI: use new `Button` component (#57668) * Use new design system Button component * Move presentation toggle into container * Use new Button in submit area * Remove inline styles * Add edit icon commit 268df9391077a054cd6478aa8af2274ddf5600f5 Merge: 69f7890d8cb 1e936cd4732 Author: Seth Nickell <snickell@gmail.com> Date: Wed Apr 3 09:48:03 2024 -0700 Merge pull request #57761 from code-dot-org/staging DTT (Staging > Test) [seth] commit 1e936cd47328407cc154d7cc457dd4981ee4b723 Merge: e4741992140 45da6889780 Author: Seth Nickell <seth.nickell@code.org> Date: Wed Apr 3 09:46:59 2024 -0700 Merge pull request #57760 from code-dot-org/P20-829/prepare-units-on-demand P20-829: Fix `ScriptLevelsControllerTest` commit 45da68897809ea020575dd4e285122a40d02c92e Author: Artem Vavilov <artem.vavilov.7@gmail.com> Date: Wed Apr 3 19:42:52 2024 +0300 P20-829: Fix `ScriptLevelsControllerTest` commit e4741992140657d6b4f90961913f9a6b4fdee5bb Author: Dave Bailey <davidsbailey@users.noreply.github.com> Date: Wed Apr 3 09:31:46 2024 -0700 Fix UI status for pending requests to evaluate entire class (#57634) * dependent destroy learning_goal_ai_evaluations * include SharedConstants * extract last_attempt_evaluated * fix attempted_unevaluated_count and add pending_count * fix logic for pending evals in status-all response handler * unskip uitest to eval entire class * improve comments * add delay in TestAiProxyController * also check pending count in existing rails tests * add new rails tests for pending count * prefer RUBRIC_AI_EVALUATION_STATUS over raw numbers * improve test setup readability * fix js refs to STATUS_ALL * js test shows pending status when eval is pending * remvoe stray console logs * Revert "add delay in TestAiProxyController" This reverts commit 06e931fbfd75b1b8095d9ffc633e2c3f4395615e. commit 5c8116e47d7ca453388b6b6ca98fee7c41bc4eca Author: Dave Bailey <davidsbailey@users.noreply.github.com> Date: Wed Apr 3 09:14:10 2024 -0700 manual DTS (Levelbuilder > Staging) update focus-on-coding3-2023 Lesson 14 rubric (#57758) * levelbuilder content changes (dave) - add rubric to focus-on-coding3-2023 Lesson 11 * levelbuilder content changes (dave) - fix focus-on-coding3-2023 Lesson 14 rubric --------- Co-authored-by: Continuous Integration <dev@code.org> commit d76b4564ecee7ec477beb6c60b222674ba717dcc Author: Liam Frye-Mason <liam.fryemason@code.org> Date: Wed Apr 3 08:45:30 2024 -0700 Add component library to eslint import order rule (#57690) commit 6a522ca64067a3e60348100698b1257d231ddfbb Author: Kelby Hawn <9256643+kelbyhawn@users.noreply.github.com> Date: Wed Apr 3 08:04:19 2024 -0700 Add static recommended PL to the bottom of the My PL page (#57730) * create ActionBlocksWrapper.jsx * create ActionBlocksWrapper.story.jsx * update storybook description * fix justify spacing on oneColumnActionBlock * remove unneeded image class * update for loop variable name to be more descriptive * add strings * refactor LandingPage.jsx and add ActionBlocksWrapper * update landingPageTest.js * add top margin to the new action blocks section * add data-testid attribute to ActionBlocksWrapper.jsx * update test to use getByText commit 6d1075a6d8a53f10485b9bd16f8415d752729a01 Merge: b961962c184 ce6fd898209 Author: Stephen Liang <stephenliang@users.noreply.github.com> Date: Wed Apr 3 07:57:17 2024 -0700 Merge pull request #57692 from code-dot-org/stephen/lti-co-teacher feat(lms): disable adding or removing co-teachers if the section is managed by lms commit c991eee728d580b8a887cc6ffaccdcc12063fe2d Author: Continuous Integration <dev@code.org> Date: Wed Apr 3 13:55:53 2024 +0000 levelbuilder content changes (dave) - fix focus-on-coding3-2023 Lesson 14 rubric commit b961962c184401b3acf8937103d523001c5e7686 Author: Dave Bailey <davidsbailey@users.noreply.github.com> Date: Wed Apr 3 06:04:23 2024 -0700 levelbuilder content changes (dave) - add rubric to focus-on-coding3-2023 Lesson 11 (#57753) Co-authored-by: Continuous Integration <dev@code.org> commit 78f8bbeaaa9c8b81f8d5f5357f290920d15d7caf Author: Continuous Integration <dev@code.org> Date: Wed Apr 3 13:02:57 2024 +0000 levelbuilder content changes (dave) - add rubric to focus-on-coding3-2023 Lesson 11 commit cd85f4830fa9e56dcf1856aca066bd63dc392fca Merge: 00c22ebee7a fb8c4b42dea Author: Code.org Deploy Systems <dev@code.org> Date: Wed Apr 3 00:35:31 2024 -0700 Merge pull request #57751 from code-dot-org/dts_candidate_2024-04-03 DTS (Levelbuilder > Staging) [robo-dts] commit fb8c4b42dea16110f78255b358f05741d7a943c0 Merge: 00c22ebee7a 8b4bb7f3b50 Author: Continuous Integration <dev@code.org> Date: Wed Apr 3 07:35:24 2024 +0000 Merge remote-tracking branch 'origin/levelbuilder' into dts_candidate_2024-04-03 commit 1c35998f5785283b277ba8b33c8219c9acf127a5 Author: breville <brendan@code.org> Date: Wed Apr 3 07:13:04 2024 +0000 Panels: two more layouts commit 91995af8beb2f238fa8c2fd34c16bc3f62f4692f Author: Continuous Integration <dev@code.org> Date: Wed Apr 3 07:09:28 2024 +0000 hoc i18n updates commit 6cdcbac04dace983b8f49255ba377ef4f9fc1627 Author: Continuous Integration <dev@code.org> Date: Wed Apr 3 07:09:18 2024 +0000 apps i18n updates commit 6766ad0b1ffe7a8afa9ef753a86fad830255b8fa Author: Continuous Integration <dev@code.org> Date: Wed Apr 3 07:09:08 2024 +0000 dashboard i18n updates - Chinese Traditional commit dbc02cc7df97069b23028f41f0fb39eb8cedfe66 Author: Continuous Integration <dev@code.org> Date: Wed Apr 3 07:09:04 2024 +0000 dashboard i18n updates - Chinese Simplified commit e3a2033b201c0bb84aedcc79f6aaeb860e5932b2 Author: Continuous Integration <dev@code.org> Date: Wed Apr 3 07:09:01 2024 +0000 dashboard i18n updates - Vietnamese commit 02eaf70e73c74ccd33fd4302933bd2be35dd0bde Author: Continuous Integration <dev@code.org> Date: Wed Apr 3 07:08:57 2024 +0000 dashboard i18n updates - Uzbek commit 3c3c880fb343eac0203f8396cd364b511ac90f18 Author: Continuous Integration <dev@code.org> Date: Wed Apr 3 07:08:50 2024 +0000 dashboard i18n updates - Ukrainian commit 6c1b1bdca4a47d6f0742a831b70631a09b7c2573 Author: Continuous Integration <dev@code.org> Date: Wed Apr 3 07:08:47 2024 +0000 dashboard i18n updates - Turkish commit 633ceb847747485d4455b87e5b2b8625144fc0a3 Author: Continuous Integration <dev@code.org> Date: Wed Apr 3 07:08:43 2024 +0000 dashboard i18n updates - Thai commit 8eec94365401e572d27f9655ea983628f341a0b3 Author: Continuous Integration <dev@code.org> Date: Wed Apr 3 07:08:33 2024 +0000 dashboard i18n updates - Tamil commit 8885d9304b9ce9959d4c0e86336fd446494fdf91 Author: Continuous Integration <dev@code.org> Date: Wed Apr 3 07:08:30 2024 +0000 dashboard i18n updates - Swedish commit da51d4f895a1d59c4349da3650e7bf29a99fb58e Author: Continuous Integration <dev@code.org> Date: Wed Apr 3 07:08:23 2024 +0000 dashboard i18n updates - Albanian commit 4502ea56c024f1765c7e61db5c2022f61e73cac1 Author: Continuous Integration <dev@code.org> Date: Wed Apr 3 07:08:16 2024 +0000 dashboard i18n updates - Slovak commit 58c7d509af51992734ee957342620760146605fb Author: Continuous Integration <dev@code.org> Date: Wed Apr 3 07:08:04 2024 +0000 dashboard i18n updates - Russian commit e565054aa37b1396ecd5ff78aa8a60eae83702bb Author: Continuous Integration <dev@code.org> Date: Wed Apr 3 07:08:00 2024 +0000 dashboard i18n updates - Romanian commit 74f58e6aa522e66540bf6132fb34d21ba0ac88fb Author: Continuous Integration <dev@code.org> Date: Wed Apr 3 07:07:56 2024 +0000 dashboard i18n updates - Portuguese commit e32e2a213b3de6bbfd858773c8622fada456de1d Author: Continuous Integration <dev@code.org> Date: Wed Apr 3 07:07:47 2024 +0000 dashboard i18n updates - Polish commit d477fce4f25aeac0ff7fb4e66aba51664baaf18c Author: Continuous Integration <dev@code.org> Date: Wed Apr 3 07:07:43 2024 +0000 dashboard i18n updates - Norwegian commit f9931b9d3f558bbef15f3d44d0bd4b78d8d2ab97 Author: Continuous Integration <dev@code.org> Date: Wed Apr 3 07:07:36 2024 +0000 dashboard i18n updates - Dutch commit 6318b8e8311481fae1b1467b5d317c4d7997ebb1 Author: Continuous Integration <dev@code.org> Date: Wed Apr 3 07:07:24 2024 +0000 dashboard i18n updates - Malay commit 05cd6273d5479dcc66c8bb5f3b5242b93b7091bc Author: Continuous Integration <dev@code.org> Date: Wed Apr 3 07:07:17 2024 +0000 dashboard i18n updates - Mongolian commit 0563cf5985d411613f05a8128b3a25df5fe94fc5 Author: Continuous Integration <dev@code.org> Date: Wed Apr 3 07:06:56 2024 +0000 dashboard i18n updates - Korean commit 5126d98fbabcd41ff840beaf29bfa01d469c7ea9 Author: Continuous Integration <dev@code.org> Date: Wed Apr 3 07:06:49 2024 +0000 dashboard i18n updates - Kannada commit a42aac131efa29c856fcbe6beea3a668f8ddda81 Author: Continuous Integration <dev@code.org> Date: Wed Apr 3 07:06:46 2024 +0000 dashboard i18n updates - Kazakh commit f373155193fe5f79462e5703d19ad21c8dbc1b1f Author: Continuous Integration <dev@code.org> Date: Wed Apr 3 07:06:42 2024 +0000 dashboard i18n updates - Georgian commit 8b76730876c2a4d3b8e24cf1b390854b4613ff34 Author: Continuous Integration <dev@code.org> Date: Wed Apr 3 07:06:38 2024 +0000 dashboard i18n updates - Japanese commit 26216f56f6d9da6300eadcddf03b40891ea50696 Author: Continuous Integration <dev@code.org> Date: Wed Apr 3 07:06:35 2024 +0000 dashboard i18n updates - Italian commit c57c464833999f708f25f2dd2ef495dc57f768ef Author: Continuous Integration <dev@code.org> Date: Wed Apr 3 07:06:31 2024 +0000 dashboard i18n updates - Icelandic commit 11c1fd7cab7b058617708672d55a90d2cb10d830 Author: Continuous Integration <dev@code.org> Date: Wed Apr 3 07:06:24 2024 +0000 dashboard i18n updates - Indonesian commit 9f049ad56545d2e806fb81d02ea4f09defc624af Author: Continuous Integration <dev@code.org> Date: Wed Apr 3 07:06:18 2024 +0000 dashboard i18n updates - Hungarian commit ba8f2fcdc7120cd8cb80c644de83ec69e8b87f4d Author: Continuous Integration <dev@code.org> Date: Wed Apr 3 07:06:11 2024 +0000 dashboard i18n updates - Hindi commit a51f341284b5806a19d7fc8a5fa17c5185afae4a Author: Continuous Integration <dev@code.org> Date: Wed Apr 3 07:06:07 2024 +0000 dashboard i18n updates - Hebrew commit aaacf5eb5d4a76169b78b731bae5e2886db49b88 Author: Continuous Integration <dev@code.org> Date: Wed Apr 3 07:05:55 2024 +0000 dashboard i18n updates - French commit 526dfca18176e04dcc633e1a3fb09262914370e8 Author: Continuous Integration <dev@code.org> Date: Wed Apr 3 07:05:51 2024 +0000 dashboard i18n updates - Filipino commit 76c74bfd96f0a4366538ce3f3d0b3977703fcce0 Author: Continuous Integration <dev@code.org> Date: Wed Apr 3 07:05:44 2024 +0000 dashboard i18n updates - Persian commit 377a2bf03d3b8a0f02db4fce0b38407490885d23 Author: Continuous Integration <dev@code.org> Date: Wed Apr 3 07:05:31 2024 +0000 dashboard i18n updates - Spanish, Mexico commit 23c1780d0201ae8ec2c6c5e3e65409f8d2b9472e Author: Continuous Integration <dev@code.org> Date: Wed Apr 3 07:05:27 2024 +0000 dashboard i18n updates - Spanish commit fadf0be3e60d4bfe3f4341758156e522f4807583 Author: Continuous Integration <dev@code.org> Date: Wed Apr 3 07:05:21 2024 +0000 dashboard i18n updates - Greek commit 7911463fc1e00cf431a30003f45b264d223d0ee8 Author: Continuous Integration <dev@code.org> Date: Wed Apr 3 07:05:17 2024 +0000 dashboard i18n updates - Dhivehi commit f1d92d63ee6429f35151f9949a3a7060fdc67885 Author: Continuous Integration <dev@code.org> Date: Wed Apr 3 07:05:13 2024 +0000 dashboard i18n updates - German commit 712998eb9cdae9a60962d22f20e27cfd47184112 Author: Continuous Integration <dev@code.org> Date: Wed Apr 3 07:05:06 2024 +0000 dashboard i18n updates - Czech commit dd544e43020e09190ff0dc0778e55d7de1a417a7 Author: Continuous Integration <dev@code.org> Date: Wed Apr 3 07:04:59 2024 +0000 dashboard i18n updates - Catalan commit de87c2c6a87e628b2cfedee6c5753dbdc9459495 Author: Continuous Integration <dev@code.org> Date: Wed Apr 3 07:04:47 2024 +0000 dashboard i18n updates - Azerbaijani commit af403cc135facd5ec36da236e5952698b1c4dcad Author: Continuous Integration <dev@code.org> Date: Wed Apr 3 07:04:43 2024 +0000 dashboard i18n updates - Arabic commit 80cc031711a2d5e24679ccd9cabb0a7a6b343e5f Author: Continuous Integration <dev@code.org> Date: Wed Apr 3 07:04:36 2024 +0000 pegasus i18n updates commit 21eb9c0040461338f3386de459d59d3cfef13632 Author: Continuous Integration <dev@code.org> Date: Wed Apr 3 07:04:32 2024 +0000 etags updates commit 69f7890d8cb432020e47e29789d8b6337c7d7a0e Merge: 68824225020 00c22ebee7a Author: Code.org Deploy Systems <dev@code.org> Date: Tue Apr 2 23:52:09 2024 -0700 Merge pull request #57748 from code-dot-org/staging DTT (Staging > Test) [robo-dtt] commit 00c22ebee7a619d9fc958511e5accfcadbbab000 Merge: 9039c81b7c4 0cdec6c4ffd Author: Brendan Reville <breville@users.noreply.github.com> Date: Wed Apr 3 14:24:16 2024 +1100 Merge pull request #57534 from code-dot-org/music-one-pack commit 9039c81b7c4793f2d14a9517519f2ec44ba3cead Author: Artem Vavilov <artem.vavilov.7@gmail.com> Date: Wed Apr 3 06:13:13 2024 +0300 P20-829: Prepare Units only when needed on the Dashboard test run (#57743) * P20-829: Prepare Units only when needed on the Dashboard test run * P20-829: Fix failed tests * P20-829: Improve the method name commit 0cdec6c4ffde6f702a0631181b73ea9e6dd8548f Author: breville <brendan@code.org> Date: Wed Apr 3 00:17:15 2024 +0000 New parameter commit cf805a0e720f0c35f4d0a8b00c3633a59f71b3c3 Author: Sanchit Malhotra <85528507+sanchitmalhotra126@users.noreply.github.com> Date: Tue Apr 2 20:07:02 2024 -0400 Add support for localizing Panels levels (#57667) * Add support for localizing Panels levels * add tests * include panels factory commit 83ab1a3af30c7dfc82520c04d09edb85c26ce1b5 Author: Elijah Hamovitz <elijahhamovitz@gmail.com> Date: Tue Apr 2 16:28:09 2024 -0700 Re-sign `.drone.yml` After Update (#57746) Follow-up to https://github.com/code-dot-org/code-dot-org/pull/57557, in which I updated `.drone.yml` but forgot to update the signature before merging. commit e728bef87ab366863bf0a41bfacfdc7ea4d364f0 Merge: fefa5dde9de d47c3623d26 Author: breville <brendan@code.org> Date: Tue Apr 2 23:14:58 2024 +0000 Merge branch 'staging' into music-one-pack commit fefa5dde9deab22d2fd549069f57f7e472697245 Author: breville <brendan@code.org> Date: Tue Apr 2 23:04:40 2024 +0000 Update style commit d47c3623d268bae79e05ac535693fdf9b6ac284b Author: Ken Akiha <95503833+kakiha11@users.noreply.github.com> Date: Tue Apr 2 16:02:32 2024 -0700 Ka ai tutor available component logic (#56944) * commit to switch branches * switching branches * Updated naming for clarity * Updated fields in show.js for consistency * PR feedback on variable names commit a5dfb8211a3aaf714f41c2a0ed4114ba26fdeefa Author: Hannah Bergam <hannahbergam@gmail.com> Date: Tue Apr 2 16:00:46 2024 -0700 adding statsig to finish sign up events and account settings page (#57703) commit 2d106c9362b0f7922de23e5adbbd1771af7047f6 Merge: 465407f20f0 61be4800b0a Author: Brendan Reville <breville@users.noreply.github.com> Date: Wed Apr 3 09:53:21 2024 +1100 Merge pull request #57673 from code-dot-org/music-embed Music: embeddable mini-player commit 465407f20f0be6e24600740937e990bdc154fc9e Author: Artem Vavilov <artem.vavilov.7@gmail.com> Date: Wed Apr 3 01:50:49 2024 +0300 P20-828: Lazy load i18n files on the Dashboard test env (#57661) * P20-806: Lazy load i18n files on test env * P20-806: Add more comments * P20-806: Apply lazy loading except in CI env * P20-828: Checks if a filename is named in correspondence to the translations it loaded * P20-806: Fix desc * P20-828: Fix comments * P20-828: Update `assert_file_named_correctly!` comments * P20-828: Memoize `valid_locales` * P20-828: Fix comments commit 8589ddd26d956552294deea62e4f1916ac06c0a0 Author: Mike Harvey <43474485+mikeharv@users.noreply.github.com> Date: Tue Apr 2 18:41:52 2024 -0400 loopHighlight (#57738) commit 688242250202259dd3a38572db73c0c130663a72 Merge: 50180ce98e4 3a75ab452a9 Author: Code.org Deploy Systems <dev@code.org> Date: Tue Apr 2 15:22:08 2024 -0700 Merge pull request #57741 from code-dot-org/staging DTT (Staging > Test) [robo-dtt] commit 3a75ab452a9a141e8c1fa367fb0a1bfb4462cd7b Merge: 2d102f7fb0d 09db51c00a4 Author: Mario Gil Correa <66776217+mgc1194@users.noreply.github.com> Date: Tue Apr 2 15:04:33 2024 -0700 Merge pull request #57685 from code-dot-org/i18n-sync-in-up-04-01-2024 I18n sync In & Up 04/01 commit 2d102f7fb0d1aeb6096646c33640d2c7b49c105e Author: Elijah Hamovitz <elijahhamovitz@gmail.com> Date: Tue Apr 2 14:43:08 2024 -0700 Revert "Revert "Update Redis gem to 4.8.1"" (#57557) * Revert "Revert "update redis gem to 4.8.1 (#57204)" (#57556)" This reverts commit 308f47596df8303c00647242fab43aeeb8ff5193. * Provide functional default for `CDO.geocoder_redis_url` * replace deprecated Redis.connect invocation with recommended Redis.new * install redis in docker so we can test actual redis functionality there * actually start redis server after installation * Revert "actually start redis server after installation" This reverts commit ee6f63db0fcc2149a5b1e35deccf16d6df1ae14b. * Revert "install redis in docker so we can test actual redis functionality there" This reverts commit a74c4e902a497c1684d33e0ca1a11c2ab9a1ed8a. * add redis service to unit-tests to mirror the one that exists in ui-tests * mirror docker-compose change in drone config * intentionally revert to using deprecated method, to verify that tests will catch * Revert "intentionally revert to using deprecated method, to verify that tests will catch" This reverts commit 0c0ad1c96cfc1cae06f8012063bd3c8838b7ec4d. commit d9ff1fbfc92ff9ef63960c2ee3869529752cd549 Author: Elijah Hamovitz <elijahhamovitz@gmail.com> Date: Tue Apr 2 14:33:32 2024 -0700 Enable eslint `import-order` for Netsim (#57650) * autocorrect import-order violations in netsim JS * manually fix up some import-order violations that autocorrection couldn't handle * manually remove unnecessary type annotation commit a815c77397ea498b487c3185ae4e114cd00bbb84 Author: Sanchit Malhotra <85528507+sanchitmalhotra126@users.noreply.github.com> Date: Tue Apr 2 17:16:36 2024 -0400 Rework AI Chat types (part 1) (#57698) * Rework AI Chat types (part 1) * PR feedback * AiChat -> Aichat commit ce6fd8982092e43218035f522dde672962a16285 Author: Stephen Liang <538214+stephenliang@users.noreply.github.com> Date: Tue Apr 2 13:43:41 2024 -0700 feat(lms): do not disable co-teacher settings if roster sync is disabled commit f58600ac71ea0afc71cc2648d5553bd597deeda4 Author: carl-codeorg <131809324+carl-codeorg@users.noreply.github.com> Date: Tue Apr 2 14:09:51 2024 -0700 Update LTI sync summary window (#57695) Part one of updating the summary window to support section owner disambiguation during LTI sync commit fc9e4f376c52db8fa6eff2bfd75e400fe6596600 Author: Kelby Hawn <9256643+kelbyhawn@users.noreply.github.com> Date: Tue Apr 2 13:38:43 2024 -0700 Create ActionBlocksWrapper component (#57705) * create ActionBlocksWrapper.jsx * create ActionBlocksWrapper.story.jsx * update storybook description * fix justify spacing on oneColumnActionBlock * remove unneeded image class * update for loop variable name to be more descriptive * update grid comment and add 6 tiles to Storybook commit d07f871b0345d9f8ed7da2643eaf1867c1684a82 Author: suresh <suresh@code.org> Date: Tue Apr 2 13:26:40 2024 -0700 Grant Contributors access to AI rubric. (#57731) commit a10a029ea2dcc41baa3a361badf7a0860895a325 Author: Turner Riley <56283563+TurnerRiley@users.noreply.github.com> Date: Tue Apr 2 13:23:55 2024 -0700 New school and school district data ingestion from NCES for 2022-2023 (#57659) * setup district import * update school import * add comment commit 8b88925dc2628d686cd76d06944bef60478c4668 Author: Mike Harvey <43474485+mikeharv@users.noreply.github.com> Date: Tue Apr 2 16:10:33 2024 -0400 [Google Blockly] Generate code for block array (#57719) * blocksToCode * fix linting error * re-add opt_typeFilter commit 50180ce98e4e416b98b3886509ffba43d51bddef Merge: 4e869f0401d 4caf88160a3 Author: Code.org Deploy Systems <dev@code.org> Date: Tue Apr 2 13:10:11 2024 -0700 Merge pull request #57736 from code-dot-org/staging DTT (Staging > Test) [robo-dtt] commit 4caf88160a3cc9762210a673634f2243e8fa1e20 Author: cearachew <ceara.chewning@code.org> Date: Tue Apr 2 16:01:27 2024 -0400 Ceara/aitt 566 ta fit finish (#57694) * Add min-height to the evidence levels to prevent jitter on hover, remove gap between description and tips to avoid too much space * Save draggable FAB position to session storage to keep it in the same position on a reload commit 926b55fb53c535a2756f23b351b9a7bc285d65a9 Author: Stephen Liang <538214+stephenliang@users.noreply.github.com> Date: Mon Apr 1 11:48:54 2024 -0700 feat(lms): disable adding or removing co-teachers if the section is managed by lms When the roster is managed externally by an LMS, adding or removing co-teachers will result in a desynchronized state which will lead to some confusion when a roster sync occurs. Instead, inform the teacher to make updates on the LMS directly. commit e6dfee46efa3a068c7cd07804ec996b88e59d8c6 Author: Seth Nickell <snickell@gmail.com> Date: Tue Apr 2 12:42:04 2024 -0700 Remove 'COVID-19 Cases per Country' from DatasetsController::LIVE_DATASETS commit a4d70ca9558c65f4ba1bb5ea53262b431b6256cc Author: Seth Nickell <snickell@gmail.com> Date: Tue Apr 2 12:33:33 2024 -0700 Don't try to run broken update for 'COVID-19 Cases per Country' Co-authored-by: Cassi Brenci <cnbrenci@users.noreply.github.com> commit 4e869f0401d819f6939fef5badb56dc37b96a40c Merge: 8bfc6da2aba 3a2878f3f20 Author: Code.org Deploy Systems <dev@code.org> Date: Tue Apr 2 12:12:09 2024 -0700 Merge pull request #57733 from code-dot-org/staging DTT (Staging > Test) [robo-dtt] commit 3a2878f3f20908a67e4020b8ffe044b362e2940b Author: Elijah Hamovitz <elijahhamovitz@gmail.com> Date: Tue Apr 2 11:58:04 2024 -0700 Provision new Redis-based Session Store (#57558) * Provision new Redis-based Session Store The first step in our efforts to transfer user session data from cookies to a centralized data store is to provision the data store in Cloudformation, so we can begin writing data to it. We have quite a few options for how exactly we want to provision this, but broadly speaking we can choose either Serverless or On-Demand nodes, and if we choose On-Demand we also must choose both the size and quantity of nodes we want. According to Google Analytics, we've had 28M users over the last 90 days, and 84M over the last year, which works out to between 45 and 60 million users on average over an arbitrary 200-day period (200 days being our current session duration configuration). At ~300 bytes per session on average, that's 13.5-18 GB of data we'd ideally like to store. Storing 18GB of data in Severless ElastiCache would cost ~$1620/month; even 13GB would still be ~$1170/month. Alternatively, we could use three of the smallest memory-optimized cache node (cache.r7g.large) and get 39.21GB of storage for ~$480/month, or even just use two for 26.14GB of storage at ~$320/month. An even more targeted solution could be to use a larger number of smaller nodes; 6 cache.t4g.medium nodes would allow us to store 18.5GB for ~$280/month. Throughput is less of an issue for us than storage. According to CloudWatch, we average ~800M requests per week, 15M/hour, and under 4,200/second. At an average of 300 bytes stored per session and one GET plus one PUT per request, that's 2.5 Gigabits of needed network performance for on-demand nodes (well under the 12.5 promised by cache.r7g.large nodes), or ~$25/month in ECPUs for a Serverless model. This PR proposes we provision a single cluster consisting of three cache.r7g.large nodes, with encryption, automatic failover, and multi-zone availability all enabled. That should give us plenty of room for growth without too much unnecessary expense. The next step in our broader efforts to enable this feature is to begin writing data to redis so we can get a more accurate idea of actual usage, at which point we can revisit this decision. * move session store cloudformation configg into its own component * specify protocol in redis url config * return session_store component to the 'if frontends' block commit 800c9f69e6e9955751390d6fc27e41dd35aa4b09 Merge: 2b6d05c7947 f8a1d27c4d5 Author: Seth Nickell <seth.nickell@code.org> Date: Tue Apr 2 11:45:03 2024 -0700 Merge pull request #57672 from code-dot-org/datablock/warn-on-invalid-csv Raise `StudentFacingError` when importing an invalid CSV file commit 2b6d05c7947f54c4b6634d6f0b2d36ee23c5c1f6 Merge: c69a0af7175 6bdb829bbab Author: Seth Nickell <seth.nickell@code.org> Date: Tue Apr 2 11:41:20 2024 -0700 Merge pull request #57684 from code-dot-org/datablock/find-shared-table-exceptions Raise an exception if find_shared_table() doesn't exist commit c69a0af7175b6f0e0e4c60770f854e45e570962e Merge: 03cadc27c81 e90f391e93a Author: Seth Nickell <seth.nickell@code.org> Date: Tue Apr 2 11:39:27 2024 -0700 Merge pull request #57683 from code-dot-org/datablock/retry-table-or-create Retry table_or_create on failure commit 8bfc6da2aba1b1cc8bee324b3243aa9fc9936b0b Merge: 71ab483734d 03cadc27c81 Author: Code.org Deploy Systems <dev@code.org> Date: Tue Apr 2 11:24:09 2024 -0700 Merge pull request #57729 from code-dot-org/staging DTT (Staging > Test) [robo-dtt] commit 03cadc27c81262a832c03095cdcf008e4ec4f91f Author: Cassi Brenci <cassi.brenci@code.org> Date: Tue Apr 2 11:23:43 2024 -0700 Remove null ref from datablock_storage debug view (#57630) commit 8b4bb7f3b5055b848e51721ea841a463a21eb25d Merge: 30f7b887804 ddba3e408f5 Author: Turner Riley <56283563+TurnerRiley@users.noreply.github.com> Date: Tue Apr 2 10:40:51 2024 -0700 Merge pull request #57727 from code-dot-org/dtl_candidate_ddba3e40 DTL (Test > Levelbuilder: ddba3e40) commit 1b645b0178a9173b48c4a02320ccfe41ee7d5b2a Author: Kaitie O <kaitie@code.org> Date: Tue Apr 2 12:25:52 2024 -0500 added experiment flag for new dashboard view (#57701) * added experiment flag * moved location of flag check commit f9bbcb828779d3373f38f1c2dd639…
Warning!!
The AP CSP Create Performance Task is in progress. The most critical dates are from April 3 - April 30, 2024. Please consider any risk introduced by this PR that could affect our students taking AP CSP. Code.org students taking AP CSP primarily use App Lab for their Create Task, however a small percent use Game Lab. Carefully consider whether your change has any risk of alterering, changing, or breaking anything in these two labs. Even small changes, such as a different button color, are considered significant during this time period. Reach out to the Student Learning team or Curriculum team for more details.
Description
Adds amplitude/statsig logging to various events to track the time between student submitting/ teacher pressing evaluate and teachers looking at ai evaluations. Adds the following events:
Links
https://codedotorg.atlassian.net/browse/AITT-565
https://codedotorg.atlassian.net/browse/AITT-543
Testing story
Adds unit tests for the UI amplitude/ statsig logging, but I couldn't figure out how to add a test for the logging in the ruby tests/ couldn't find any other examples of testing the statsig logging in ruby.
PR Checklist: