Hey @btriller,
Good catch regarding the function comment. Thanks!
Regarding the docs, I think it's best that the param
dlg_flag
stay but with maybe a small description ie,deprecated from version 6.x. See dlg_manage()
. This would inform the users to look for the alternative and better option ofdlg_manage()
.What do you think?
I only realized that not removing module parameter in docs was intended after opening this PR. Feel free to close this PR.
Still, IMHO it'd be better to remove this in devel docs and add removal to 6.0 upgrade docs.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.