What ever works for you is the right way. There are numerous popular ways of organizing, but they all require keywording to make the best use of the database. I'm lazy and have not always done keywording carefully.
If you do not religiously keyword everything, organize the way you feel is comfortable.
BUT ... Lightroom knows the year, month, and day of each photo, merely go to the filter panel in the library and the first column is the date. Drill down to the date you want, and only images from that period are shown, it could be year, year and month, or year, month, and day.
This might tell you that filing them by location, event, or some other type of organization would provide information that Lightroom does not do (unless you keyword those things)
I file mine by event, the date takes care of itself. If I were to always keyword them, they could all go into one directory. I also break my rule for a multi day event, adding the date to the directory such as when I'm doing a play, and there are two or more days of dress rehearsals, then I create a folder for each day because some of the actors might be missing or out of costume on one of the days and I do not want to miss his/her image. Of course, I'll forget this after a month or two, but by then, the project is finished.