Closed thiagomayllart closed 3 years ago
Looks pretty good! I really only have one gripe, which is the structure of the rportforward
data. Being a series of 5 nested dictionaries is gonna make it hard for other maintainers to read and understand what data is supposed to be fed into the variety of functions that beast is thrown into. Thanks again for your time and effort on this!
Perfect!! I'll be making the modifications as soon as possible!! Thank you again for the response 😃
Hi @djhohnstein !! I've just update the structure of rportforward data and task.display_args. I think it should be fine now 😃
Hello everyone! I've recently updated the port forward module to fit the 2.2 updates. It is working the same way as the previous pull request in: #18