feat(lockup, flow): add transferFromPayable

[andreivladbrg]

closes #1376

i implemented transferFromPayable instead because the compiler doesn’t allow adding override and the payable specifier to a non-payable function:

     Error: Compiler run failed:                                                                                       
     Error (6959): Overriding function changes state mutability from "nonpayable" to "payable".                        
        --> src/SablierFlow.sol:527:5:                                                                                 
         |                                                                                                             
     527 |     function transferFrom(address from, address to, uint256 tokenId) public payable override {              
         |     ^ (Relevant source part starts here and spans across multiple lines).                                   
     Note: Overridden function is here: 

note: i added the function to lockup as well because i think it’s good for consistency. lmk if you agree or if you consider the increased size a problem (it’s still below the margin)

[andreivladbrg]

lmk if you think we only need one branch, or if keeping both is ok

also, do you think we should use transferFromPayable in the batch tests? IMO these are sufficient

[andreivladbrg]

@smol-ninja the fork tests seem to be failing. i tested locally, and this is not related to this PR, as it failed in staging as well. let’s fix it in another PR

[smol-ninja]

@andreivladbrg the fork tests are fixed in #1366. See 2539c05