add invalid and error details to vcard import report

[johndoh]

for #9591

currently there are 2 silent error cases on vcard import, invalid (fails roundcube contact validation) and error (its valid but i didnt insert for some reason e.g. plugin prevented it) this adds messages about those to the screen.

[pabzm]

Thank you for the contribution! I really appreciate your attention to issues related to "your" topic!

[pabzm]

Looks good to me, thank you!

[alecpl]

Approved with some comments.

[alecpl]

We should probably get rid of HTML tags from all these three messages.

[johndoh]

I think removing the <b> tags would be a breaking change because the formatting should be moved to the skin to do this properly. I can work on this in a separate PR.

[pabzm]

@alecpl I'll merge this in one week if nothing happens until then, since you left your approval already and the only open topic is very minor.

[alecpl]

Feel free to merge, close the related ticket (assigning milestone) if appropriate and update the changelog.

[pabzm]

@johndoh Would you add a line to the Changelog? (I can do it as well but would prefer to have it as part of the PR.)

[pabzm]

/remind me to merge on Monday

[github-actions]

@pabzm set a reminder for 11/25/2024

[johndoh]

I've added a changelog entry. If you don't like the text or its in the wrong place its no problem.

[github-actions]

👋 @pabzm, merge

[pabzm]

@johndoh Thanks again for your contribution! 🙏