Hi @vistothemes, I think by typographic hierarchy the reviewer meant not only the font weight, but heading sizes, paragraph sizes, line heights, etc.
I am now looking at your item - Poppins and Montserrat is a tricky combination in my opinion. Your headings font (Poppins) is clean, which I like but it’s both a strength and a weakness. When used in big headings with more than a couple of lines it doesn’t look good. If there is an option I would suggest to use a serif for the biggest headings (“display headings” if we are using Bootstrap terminology).
If there’s an option I would swap them honestly - Use Montserrat for the headings, it looks way better for headings and Poppins for the paragraph text - it’s more readable when in smaller size.
If there is no option to swap them - fix the line-height of the headings, make your lead paragraph font-size bigger.
And the most important thing - be consistent when using line-heights. In your components like cards for example in several places it needs improvement. The same goes for the footer navigation.