search

mirarr-app / mirarr

Mirarr, FOSS Movie App
Other
534 stars 16 forks source link

Added error messages #59

Closed AQMpolyface closed 1 week ago

AQMpolyface commented 1 week ago

added error messages where there was a "// Handle error" without error handling

AQMpolyface commented 1 week ago

i have removed the files you indicated. Please let me know if there's anything else that needs adjustment.

mirarr-app commented 1 week ago

@AQMpolyface thanks. But now there's some dotfiles probably from your ide. Just commit the lib folder.

AQMpolyface commented 1 week ago

Apologies for the confusion earlier, I believe I’ve done a clean fork this time with only the lib included. Would it be okay to submit a new pull request? I’m really sorry for the inconvenience.

AQMpolyface commented 1 week ago

I would understand if you dont want me to do it tho, i just made a mess for like 5 more error handling. Sorry.

mirarr-app commented 1 week ago

@AQMpolyface of course it's ok. Don't worry about it. You can always run git show HEAD~n . N being number of commits you made to see what you changed before pushing. Let me know if you need help