“I believe we have it wired right” doesn’t let us know if what you believe is true. Tell us how you have it wired and let us decide if there’s something amiss.
“I’ve checked the code” doesn’t let us know if the code is correct. Show us the program and let us see if it’s doing what you think it should be doing. We can also tell you if what you think it should be doing is actually what it needs to do.
Using controls instead of constants for the channel numbers will work if you’re going to run Robot Main. Make sure to use the popup menu to set the current value as the default before you build the project. But I think it’s simpler to use constants instead.