It's got a 90's feel to it.
Do a bit of research into modern web design trends.
Colors are weird, and layout / spacing is a bit of a mess. It's missing a sense of balance - let elements breathe and be consistent with spacing.
The issues with color really aren't your fault. I don't even bother taking projects with companies unwilling to update their logo to look good on the web.
I'm assuming this is one of your first projects, so you aren't absolutely fucked.. But this one would need work to make it acceptable
