Bug 7501 - Need to add line to moving_targets.js so it reads target_interval from the config file
Summary: Need to add line to moving_targets.js so it reads target_interval from the co...
Status: RESOLVED FIXED
Alias: None
Product: 7Gen Blocks
Classification: Unclassified
Component: Beta version (show other bugs)
Version: .01
Hardware: PC Mac OS
: --- normal
Assignee: AnnMaria De Mars
URL:
: 7502 7503 (view as bug list)
Depends on:
Blocks:
 
Reported: 2023-06-09 04:19 MDT by AnnMaria De Mars
Modified: 2023-06-11 20:42 MDT (History)
1 user (show)

See Also:
Legacy Creation Date:
Legacy ID:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description AnnMaria De Mars 2023-06-09 04:19:37 MDT
If you changed 
const targetInterval = 700; // ms

to 

let targetInterval ; // ms

and then in the initializeMovingTargets function added the third line below, 


function initializeMovingTargets() {
    targets = targets_config.targets;
    targetSlots = targets_config.target_slots;
    targetInterval = targets_config.target_interval ;


it would actually use the target_interval value from the moving_targets_config.js 

Right now, the target interval is 700ms regardless of what they put in the config file
Comment 1 Dennis De Mars 2023-06-11 20:41:44 MDT
*** Bug 7502 has been marked as a duplicate of this bug. ***
Comment 2 Dennis De Mars 2023-06-11 20:42:04 MDT
Fixed in commit 6cfec5d

There was in fact a serious bug in the intialization of this block which has now been fixed.
Comment 3 Dennis De Mars 2023-06-11 20:42:30 MDT
*** Bug 7503 has been marked as a duplicate of this bug. ***