Application crashes if id does not exist/data is desynched with user database

[LDSamson]

Creating a data.frame here will fail if the list of ids is NULL, causing the application to crash:

clinsight/R/shiny.R

Lines 2 to 6 in 7026387

	with(val, data.frame(
	id = unlist(ids),
	reviewed = ifelse(isTRUE(review), "Yes", ifelse(isFALSE(review), "No", NA_character_)),
	row_id = row_id
	))

replacing it with id = unlist(ids) %||% NA_integer would solve it here, but then it will error further downstream in mod_review_forms because ids are missing (when using dplyr::rows_update).

This occurs when you update the metadata, merge it again with data, but forget to update the application database.
@jthompson-arcus this is of course a result of the bigger issue that this desynched state is possible in the first place See this #114.

Probably good to (in order of importance):

1. better ensure that database is always up to date with the study_data
2. make the app more robust if this is not the case/give a helpful error message