Placement of Pullup Resistors on SDA/SCL Lines in 8-Slave I2C Configuration

I have a beginner question related to pullup resistors.

I have attached 8 Pro Mini slaves to my Uno master. Both are 5V boards. The Pro Minis are each being fed data from digital sensors (they output a square wave). The sensor lines are blue wires in the picture below

As configured, I am not getting any data from 7 of 8 of the slaves. I think this is due to the lack of pullups on my breadboard.

I photoshopped in orange where I think I should place the resistors. Can you tell me if this is correct?

General color coding of the wires:

-Yellow = A5 / SDA
-Green = A4 / SCL
-Blue = digital input signal from sensors
-Red = 5V
-Black = ground

I think either of the configurations will work. One picture called "Orange" and the other is "Orange2." They accomplish the same thing. One just taps into the 5V rail of the breadboard

Second picture

Both seem to be correct.

Looks OK to me (but I'm not the hardware specialist :wink: ) Do all minis have their own address on the I2C bus?