Use significant suites name + variables + keywords (you did already)
Try to add assertions at the end of each test
Add [Setup] and [teardown] for each test or test suite (depending on the dependency you have between your tests)
I used Robot framework for almost 2 years for desktop application (using sikiuli library to locate images and autoit…)
Let me know if I can help you with something ?
Looks like a great first try I have done the same course/s and it obvious you picked up everything well from that course.
I would recommend splitting out your keywords from your locators and making use of a separate variables file also. I have an example as such here, though real world ended up a bit more complicated than that.
I would suggest digesting all the excellent documentation, it provides much more detail than Brian Lambs courses and will provide you with new and exciting ways of doing things, particularly as there has been a lot of changes since those courses were made!
I really enjoyed working with Robot Framework, unfortunately I no longer do.
RobotFramework (RF) is a nice framework for automated tests, and I like its versatility with cross language support via the remote library server interface, and ecosystem of existing libraries. I haven’t had the opportunity to use it much however in my line of work.