Ajustando inconsistencias de SSO

La mayoría de los campos del perfil ya se pueden especificar durante el proceso de SSO, lo cual es muy útil. Desde el nombre de usuario y el nombre, hasta la biografía, e incluso el sitio web (se añadió el año pasado). Sin embargo, aún existen algunas pequeñas peculiaridades:

  • Parece que falta location — a primera vista, parece ser el único campo obvio que no es posible modificar mediante SSO.
  • website se puede especificar durante el SSO, pero, a diferencia de la mayoría (¿todos?) de los demás campos de perfil de SSO, no tiene una configuración del sitio correspondiente que permita sobrescribir el valor local y evitar que el usuario lo modifique.

Supongo que esto simplemente se pasó por alto a medida que la base de código crecía con el tiempo, ya que no se me ocurre ninguna razón para estas excepciones. Lamentablemente, mis conocimientos de Ruby y Discourse no son suficientes para enviar un PR, pero quería reportarlo; quizás alguien tenga la capacidad técnica para hacerlo.

He implementado los cambios para que los dos elementos anteriores funcionen. Estaría encantado de contribuirlos de nuevo a Discourse, pero no sé si hay interés en incorporarlos upstream. /cc @sam

Avísame antes de que tenga que pasar por todo el proceso para enviar el PR. El código funciona, pero podría necesitar algunas revisiones o cambios, ya que soy nuevo en Discourse.

Adelante y comienza la PR para que puedas pasar por el proceso del CLA, pero:

Esto no parece realmente valioso para molestarse en hacerlo: otros campos en esa página, como la imagen de fondo de la tarjeta de usuario, no estarán disponibles en las plataformas de SSO, por lo que no hay ninguna ventaja de consistencia al agregar un bloqueo para website.

1 me gusta

Lo siento, no estoy seguro de haber entendido lo que quisiste decir allí. Estoy de acuerdo en que el fondo de la tarjeta tiene un uso limitado en SSO (aunque eso ya está implementado). website también es un campo SSO ya implementado; solo he añadido una configuración sso_overrides_website, como las que ya existen para nombre de usuario, avatar, biografía, etc., para que sea posible no permitir cambios locales y que el valor de SSO tenga prioridad.