I know this is overdue. To be honest, I don’t like animations. The most appropriate way is displaying a skeleton. But we don’t need to make it much complex. A simple “Loading…” text seems professional than using animation for me. Just my 2 cents.
Also I saw that it was a full page animation in your Pull Request.. Adding the loader within only the loading area would better instead of displaying a full page because it breaks breaks the flow when switching between pages.
A good study : https://uxdesign.cc/what-you-should-know-about-skeleton-screens-a820c45a571a?gi=bd3aa11ba5d4
Cc: @akshika47