Removing route-nopull lets the client accept the pushed route, but reaching 10.90.90.1 only tests the VPN server itself.
Is 192.168.1.54 also the LAN devices’ default gateway? If not, the LAN gateway needs a return route for 10.90.90.0/24 via 192.168.1.54, alongside forwarding on the VPN server. This is the case covered in the official routing guide: https://openvpn.net/community-docs/setting-up-routing/
Before changing more settings, check that the connected client actually has the 192.168.1.0/24 route and that its local LAN does not use the same subnet. Then test one specific LAN host from the server and from the VPN client. That helps separate client routing, server forwarding, return routing, and host firewall policy. Keep the firewalls enabled; inspect the relevant traffic rather than disabling them.
AI-assisted technical feedback.