-
Notifications
You must be signed in to change notification settings - Fork 2
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
Maintenance release RC tests #818
Comments
Not yet ready for tests (may need rc.2s on some). Once it IS ready, I'll pass to @ariel-phet for prioritization. |
RCs are ready for testing. |
@phet-steele these tests would be good to perform sooner than later (preferably before May 1) as we would like these releases published well before publishing the android app. |
Also @ariel-phet, phetsims/vibe#30 is pretty serious so I'll more than likely get to these tests immediately. |
@phet-steele sounds good |
@jonathanolson, balancing-chemical-equations shows no change with |
Fixed balancing-chemical-equations, new RC linked in main list. |
Fixed, thanks. |
@jonathanolson Done! Everything went swimmingly. All runs included the brand, had pingver=3, and hid the appropriate items in the PhET menu with the query param. And sound worked, too. |
Ok, will be pushing out production releases. |
The following RCs are available for for https://github.com/phetsims/phet-android-app/issues/16, phetsims/phetcommon#37 and phetsims/vibe#30. I'll comment when they are fully deployed.
The following complete list should be tested for:
Basic interaction (no major introduced bugs, standard ~30sec or so)
When given the
?phet-android-app
query parameter they should includephet-android-app
as theloadType
in the immediate.gif Yotta message.I believe @phet-steele should be familiar with checking the network tab of dev tools for this.
Additionally, when that query parameter is passed, there should NOT be a "Check for Updates" item in the PhET menu, and it won't have update information in the About dialog.
Any runs should include pingver=3 (NOT 2) in the Yotta message, and the brand query parameter in it should be
acid-base-solutions
area-builder
arithmetic
atomic-interactions
balancing-act
balancing-chemical-equations
balloons-and-static-electricity
beers-law-lab
bending-light
build-an-atom
charges-and-fields
color-vision
concentration
energy-skate-park-basics
faradays-law
forces-and-motion-basics
fraction-matcher
friction
function-builder
graphing-lines
gravity-and-orbits
gravity-force-lab
hookes-law
isotopes-and-atomic-mass
john-travoltage
least-squares-regression
make-a-ten
molarity
molecule-shapes
molecule-shapes-basics
molecules-and-light
neuron
ohms-law
ph-scale
ph-scale-basics
plinko-probability
reactants-products-and-leftovers
resistance-in-a-wire
rutherford-scattering
states-of-matter
states-of-matter-basics
trig-tour
under-pressure
unit-rates
wave-on-a-string
The following subset of sims have the Vibe fix, and should be tested to ensure they have proper sound on iOS (in addition, spot-checking one to ensure sound is good elsewhere):
Also BASE includes fixes for phetsims/special-ops#68, and is a continuation of #795:
The text was updated successfully, but these errors were encountered: