Your schematic shows a 4 in serie first, then a 3 in parallel, 3S4P.
There is also the 3 in parallel first, then 4 in serie, 4S3P.
In your 3SP4, take in account that if any of the battery/cell fails, the whole string in this 4S will be faulty, not supplying anymore, turning into 2S4P
In a 4S3P, if a battery fails in a 3P, the impact is less, the 2 others in the 3P go on supplying.
(Batteries are the same model, considering having the same voltage/SOC in any 3P)
Also, you could add diode insulation and fault protection, by adding diodes (good amperage) isolating the 4 (3S4P), or the 3 (4S3P), for both charging and discharging. Not very costly.
The advise is also to add fuses (good amperage) in the battery to converter, unless already in the converter setup.
If you stick with your 3S4P schematic, the batteries voltage in any string can vary independently of each other, and in the long run, it is always safe to monitor the voltage of each battery, even with a very simple battery monitor.
The green lines are outlining the idea of a "bus bar" type connection, it is better being shorter, linking each string. The negative/black line then just need one connection to this green "bus bar".