Ok update on this one. I eventually started to get some readings when I switched the I2C sensor code to the toolbox MultiI2c but only for the magnet not the accelerometer. No resistors needed and worked using the sensor shield I bought so haven't wasted money after all.
Will post the code when I get the accelerometer working!
Thanks to Paul for his suggestions and I've certainly learnt a lot more about how I2C works!