starting to add modal