If Is_Hold_Tag_Required = "Yes" Then
indicates that Is_Hold_Tag_Required is defined as a Text Field, with the literal value of Yes being entered there!
With Is_hold_tag_required being a Yes/No Field, instead of
If Is_Hold_Tag_Required = "Yes" Then
it needs to be
If Me.Is_Hold_Tag_Required = Yes Then
or, better still
If Me.Is_Hold_Tag_Required = -1 Then
Notice the Me. in front of Is_Hold_Tag_Required. This indicates to Access that you're referring to the data contained in the Control, whereas simply using Is_Hold_Tag_Required, without the Me., refers to the Control object itself.
Sometimes Access will forgive the missing Me. but not always, so it's better, IMHO, to use it.
Linq ;0)>