Skip to content

Commit

Permalink
Add assertions to remind devs that there should be a LinkedElement to…
Browse files Browse the repository at this point in the history
… the associated Property and that the Property itself should be featured, see phetsims/sun#920
  • Loading branch information
samreid authored and jessegreenberg committed Jan 10, 2025
1 parent 5f653f5 commit b2f7519
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion js/micro/model/PhotonAbsorptionModel.js
Original file line number Diff line number Diff line change
Expand Up @@ -98,6 +98,7 @@ class PhotonAbsorptionModel extends PhetioObject {
// @public
this.photonWavelengthProperty = new NumberProperty( WavelengthConstants.IR_WAVELENGTH, {
tandem: tandem.createTandem( 'photonWavelengthProperty' ),
phetioFeatured: true,
units: 'm',
validValues: [
WavelengthConstants.MICRO_WAVELENGTH,
Expand All @@ -115,6 +116,7 @@ class PhotonAbsorptionModel extends PhetioObject {
// @public (read-only) {Property.<PhotonTarget>}
this.photonTargetProperty = new Property( initialPhotonTarget, {
tandem: tandem.createTandem( 'photonTargetProperty' ),
phetioFeatured: true,
phetioValueType: EnumerationIO( PhotonTarget ),
validValues: PhotonTarget.VALUES,

Expand All @@ -136,7 +138,8 @@ class PhotonAbsorptionModel extends PhetioObject {
// @public controls play speed of the simulation
this.timeSpeedProperty = new EnumerationProperty( TimeSpeed.NORMAL, {
validValues: [ TimeSpeed.NORMAL, TimeSpeed.SLOW ],
tandem: tandem.createTandem( 'timeSpeedProperty' )
tandem: tandem.createTandem( 'timeSpeedProperty' ),
phetioFeatured: true
} );

// @public - convenience Property, indicating whether sim is running in slow motion
Expand Down

0 comments on commit b2f7519

Please sign in to comment.