That's got some things wrong with it.
Mostly the code isn't structured that way.
Teleop is supposed to be a quick execute and leave, so it can go pick up new driver orders.
The new code you added includes a While loop that doesn't ever let it leave to go get new driver commands, staying in there forever, because it's structured as a stand-alone program that's been stuck inside a sub-program. The For loops don't accomplish anything other then to slow down execution while they set the motor output over and over again to the same value.
- Device Opens on the left belong in Begin.vi (you'll have to name them there and use the new names in Teleop to reference them).
- Using the Device goes in Teleop.vi (remove the While loop and all the For loops)
- Device Closes on the right go in Finish.vi