Skip to content
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): 14a878af #31091

Merged
merged 42 commits into from Oct 3, 2019

Conversation

deploy-code-org
Copy link
Contributor

No description provided.

ajpal and others added 30 commits September 25, 2019 14:03
Long ago (in #3591) we apparently took advantage of an extra row in the cdo-state-promote gsheet that was labeled with "Sum_states" rather than a state itself.  This caused an issue in the iterator at https://github.com/code-dot-org/code-dot-org/blob/staging/pegasus/rake/generate_pdfs.rake#L21 when we attempted to generate a PDF for a state of "Sum_states".

With this change, we just do the summing in the SQL query, rather than relying upon the sum generated by the gsheet.  This will allow us to simultaneously remove the row from the gsheet.
Widget mode apps with data now work
DTT (Staging > Test) [robo-dtt]
Swap in production URL for AFE CTA
DTT (Staging > Test) [robo-dtt]
…ng-for-state-promote

Remove gsheet-based summing for state promote
ajpal and others added 12 commits October 2, 2019 16:03
[Sprite Lab] Look up Sprite by name property, not variable reference
DTT (Staging > Test) [robo-dtt]
Just a small tweak that was necessary to run the backfill script.

Followup to #31020
…dates-tweak

PL: Tweak to script to backfill old workshop dates
DTT (Staging > Test) [robo-dtt]
Assignment Updates: set up experiment, placeholder assigned on course overview
DTT (Staging > Test) [robo-dtt]
DTT (Staging > Test) [robo-dtt]
DTS (Levelbuilder > Staging) [robo-dts]
DTT (Staging > Test) [robo-dtt]
@deploy-code-org deploy-code-org merged commit cb6af71 into levelbuilder Oct 3, 2019
@deploy-code-org deploy-code-org deleted the dtl_candidate_14a878af branch October 3, 2019 22:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants