Hi Conrad,

Thanks for the advice!

I used the Worksheet_Activate event in the relevant report worksheets
instead of using the Workbook_SheetActivate, as you suggested. That
seems to be a bit more efficient as this means anyone editing the data
sheets doesn't need to be annoyed by the Workbook refreshing needlessly
all the time. I hadn't checked whether they would work as shared
workbooks, and they do indeed result in error codes when I try to share
them using RefreshAll in the code.

The more I look into this I realise it would be much easier to use
Access as a backend db and just use Excel for reporting only.

Am on a bit of a learning curve with Excel and Access! Thanks for the
help again.

Regards,
David

Brent Primary Care Trust,
NHS. UK