Could I get a second pair of eyes to check if this BMS can be used for charging?
Yes, but there are different types of balancing circuits. This one is passive (at a glance). It will burn off power if a cell (or multiple, if arranged in parallel) reach max voltage first. The downside is that if the resistor can't &quot;burn down&quot; the extra voltage, you CAN hit an overvoltage event if the cells are out of balance enough.
I'd also point out NOT to assemble it like they did in the photos. All of the (series) current is going through thin wires back to the BMS. You SHOULD use those nickel strips in between the cells, too. There isn't a &quot;functional&quot; or &quot;logical&quot; difference in assembly processes, but just be aware you could have some really bad/weird effects.
Edit: A few additional comments. If you don't know what I mean by passive/active balancing, you should read up on that and understand the differences. If you're using a simple 4S/1P config with new, matched cells, it's unlikely you'll have any real problems with a BMS like this, but if you're salvaging cells, etc., then you should pay close attention to things like capacity, current draw, cell impedance, etc.