P20-788: Prevent user creation if the associated LTI User Identity creation fails #57806
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
artem-vavilov
changed the title
LtiUserIdentity creation fails
carl-codeorg
approved these changes
Apr 5, 2024
artem-vavilov
added a commit
that referenced
this pull request
Apr 8, 2024
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…
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
Prevents user creation if the associated LTI User Identity creation fails by raising an error in the
after_createcallbackLinks