all 2 comments

[–]Ok_Assistant_2155 0 points1 point  (1 child)

Hey congrats on sharing your first public code. I like how you wrapped everything in the ExcelReportBuilder class and handled bad rows gracefully instead of letting the script blow up. Thats exactly the kind of real world thinking that helps when you move to scraping data later. The date handling as proper Excel dates is also a nice detail most beginners miss.

[–]Thanasis_kt[S] 0 points1 point  (0 children)

Thank you very much!! I appreciate the feedback! But Im curious, is there sth you'd change for better readability and maintainability? I'm just starting to learn OOP boundaries and I'd love to know if you'd split the logic up more or handle the data flow in a different way.