-
Notifications
You must be signed in to change notification settings - Fork 482
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
DTL (Test > Levelbuilder): 69949c23 #49540
Merged
Merged
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
This is one idea for a family of blocks that allows musical construction without the user knowing about variables.
DTS (Levelbuilder > Staging) [robo-dts]
…-measure Music: simple blocks idea
Music: rudimentary menu page
DTT (Staging > Test) [robo-dtt]
* Add aria labels for social media links in footer * Add aria labels to Desktop footer * Add empty alt text for logos
DTT (Staging > Test) [robo-dtt]
* pulling over relevant analytics functions from analyticsReporter in music lab * playing with how events might send * Fix EditSectionForm usage of AnalyticsReporter * Remove more code from AnalyticsReporter * Load API key from environment * Fix token -> key rename * Copy environment logic from firehose, dont send events in development, and format userId based on environment * Add test for AnalyticsReporter * Move AnalyticsReporter to be next to similar utils * Remove test event * Comment out setting user id * Update secrets key to not conflict with music lab's * Add cdo_amplitude_api_key to config.yml.erb * Log user id * Update ALWAYS_PUT to ALWAYS_SEND * Update keys in config * Add a safe access method to prevent raising Co-authored-by: Hannah Bergam <hannahbergam@gmail.com> Co-authored-by: Bethany Connor <bethany@code.org>
DTT (Staging > Test) [robo-dtt]
* show warning for calc and eval * update deprecation function * update banner text
DTT (Staging > Test) [robo-dtt]
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.
No description provided.