apple / apple/sample-cloudkit-sync-engine

var listID: String

Abierto
#10 1 comentario 0 reacciones 0 asignados Ver en GitHub
Lenguaje dominante
Swift
Estrellas
270
Forks
15
Métricas de merge de PR
Sin PR fusionados en 30 d

Descripción

### In ContactsList.swift extension Contact

`// In order for the list to update properly when fetch changes from the cloud, we need to use something other than the contact ID for the list item ID.`
`var listID: String { "\(self.id)\(Self.listIDSeparator)\(self.name)" }`

The problem with this approach is when you start adding more fields to Contact. Your approach doesn't guarantee updates when changes are fetched.

I propose changing to the following:
`var listID: String { "\(self.id)\(Self.listIDSeparator)\(self.userModificationDate)" }`

In my testing, this solves all update issues regardless of adding fields to Contact.

Thoughts?

Guía de contribución

No hay ninguna guía de contribución indexada para este repositorio

Línea de trabajo

Abre ContactsList.swift e inspecciona la extensión de Contact y la implementación actual de listID. Actualiza el identificador de la lista según lo propuesto y verifica después que los cambios obtenidos de la nube actualicen los elementos de la lista cuando cambien campos distintos del nombre del contacto.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
swift
Área
mobile-dev
Tipo de issue
Error
Dificultad
1/5
Tiempo estimado
Menos de una hora
Estado de actividad
Estancado
Claridad
Bien especificado
Aptitud para principiantes
42/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.