I think the tab order is OK here, it’s the natural order of interactive elements in the modal… if I relied exclusively on the keyboard for navigation it could be frustrating to access that link at all if the tab order was arbitrary.
Accessibility specialists recommend avoiding changing the taborder of elements on a page unless it’s absolutely necessary. Using the tabindex attribute - Vispero