need to change alert to modal