qualtRics icon indicating copy to clipboard operation
qualtRics copied to clipboard

(Soft-)deprecate `column_map()`?

Open jmobrien opened this issue 2 years ago • 2 comments

column_map() uses the old endpoint (e.g. #267), sometimes creates issues (e.g. #283), and has an established replacement (extract_colmap()). Should we go ahead and deprecate it?

If so, should we soft-deprecate w/a warning pointing to the new function, or go ahead and deprecate fully (throwing a similarly-worded error)?

Leaning towards the latter since I think the older version really doesn't offer anything uniquely useful, so most all users probably want the newer maps.

jmobrien avatar Sep 07 '22 16:09 jmobrien

Yeah, let's go straight to deprecate_warn() with this. Good idea.

juliasilge avatar Sep 07 '22 16:09 juliasilge

Okay, I was using the language inadequately here, but thanks for the link. deprecate_warn() makes sense.

jmobrien avatar Sep 07 '22 16:09 jmobrien