Here you are checking to see if each object has a property called true. It will always be false because there are no properties called true. You are only checking if online property is true. But using bracket notation or dot notation will give you access to online values in each object. Then compare if they are true. You also don’t want to return users right away. I would create a counter variable and increment it if your if statements are true then return the counter at the end of the function.