feat: Add ContactCard ViewModel, sharing and header components

[Elouan1411]

No description provided.

[Copilot]

Pull request overview

Adds contact-card state management and Compose preview components.

Changes:

• Adds card loading, editing, saving, and deletion.
• Adds QR-code header with avatar.
• Adds contact details and social-link presentation.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 4 comments.

File	Description
ContactCardViewModel.kt	Manages card state and persistence.
QrCodeHeader.kt	Renders the vCard QR header.
LinksRow.kt	Displays available link types.
ContactVCardBloc.kt	Composes the contact-card preview.

Comments suppressed due to low confidence (1)

Ui/Compose/ContactCard/src/main/kotlin/com/infomaniak/core/ui/compose/contactcard/ContactCardViewModel.kt:115

• Deleting also writes the whole User snapshot from the preview state through Room @Update. Any profile or preference changes persisted after this snapshot was loaded can therefore be reverted when only the card should change. Delete the card through an atomic card-column update rather than replacing the complete row.

            val updatedUser = current.user.copy(card = null)
            accountUtils.updateUser(updatedUser)

[tevincent]

Could be great to directly have this icon in CardLinkType.

[tevincent]

Suggested change

	Surface(
	shape = CircleShape,
	color = MaterialTheme.colorScheme.surface,
	modifier = Modifier
	.size(qrSize * 0.24f)
	.clip(CircleShape),
	) {
	Box(
	modifier = Modifier.padding(3.dp),
	contentAlignment = Alignment.Center,
	) {
	Avatar(
	avatarType = AvatarType.fromUser(user),
	modifier = Modifier
	.fillMaxSize()
	.clip(CircleShape),
	)
	}
	}
	Surface(
	shape = CircleShape,
	color = MaterialTheme.colorScheme.surface,
	modifier = Modifier
	.size(qrSize * 0.24f)
	.clip(CircleShape),
	) {
	Avatar(
	avatarType = AvatarType.fromUser(user),
	modifier = Modifier
	.padding(3.dp)
	.fillMaxSize()
	.clip(CircleShape),
	)
	}

[tevincent]

Why this is public ? It seems you only use if in the init { ... }.

[tevincent]

Instead of having a var and checking everywhere if user is null to do an early return, it would be great to just quit the VCard screen. It makes no sense to be on that screen without a user.

[tevincent]

No need for this to be public.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud