Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Change security_preferences for add apptheme #3147

Merged
merged 2 commits into from
Jan 10, 2025

Conversation

omarquisper
Copy link
Contributor

@omarquisper omarquisper commented Jan 9, 2025

ℹ️ ETA

La adicion de una dependencia requiere de varios analisis con el objetivo de que ninguna libreria impacte de forma negativa al Stack Tecnologico de Meli. Tener en cuenta que el proceso realizado por Platform Updates puede ofrecer los siguientes plazos:

Librarias Internas : 1 - 3 dias
Librarias Externas : 1 - 2 semanas / mas en casos excepcionales

Este plazo comienza una vez la solicitud esta completa y sus comentarios sean resueltos.

Descripción

Cual es la necesidad de agregar este cambio?

Estoy realizando el cambio porque el CI me está fallando, por eso estoy ajustando y cambiando a com.mercadolibre.android.security:security_preferences.

Adjunto error:

ERROR: The following dependencies are not allowed:

  • (Granular dependency not allowed for this project) com.mercadolibre.android.app_theme:core:1.2.0
Seleccione el tipo de su dependencia.
Tipo Descripción
  • Interna
Libreria/modulo desarrollado in-house en base al ecosistema de Meli.
  • Externa
Libreria desarrollada por un externo a Meli. (Google, Airbnb, otros). Requiere Ticket de Shield.
Sección obligatoria para Dependencias Externas

ℹ️ SHIELD Ticket

Link de los Tickets
Descripción Android iOS
Dependencia Externa con Artefacto en Nexus Upload an artifact Android to Repository Upload an artifact iOS to Repository
Dependencia Externa External Dependency Analysis Android External Dependency Analysis iOS
Agregue el Ticket de Shield.

Dependency Shield Ticket


Información

Seleccione las Aplicaciones impactadas por esta dependencia.

Mercado Libre Mercado Pago SmartPOS Alicia: Flex / Logistics WMS Meli Store

ℹ️ Seleccione el tipo de dependencia.

Frontend Cross

ℹ️ Mi dependencia tiene un uso controlado?

Si No

Esta dependencia agrega Artefactos (Jar, Arr, Framework, ...) o Dependencias Externas?

En caso de ser Si, agregue el Link de Nexus - Nexus Link

Si No

@omarquisper omarquisper requested a review from a team as a code owner January 9, 2025 17:35
@omarquisper omarquisper changed the title Change group for add apptheme Change security_preferences for add apptheme Jan 9, 2025
@omarquisper
Copy link
Contributor Author

@abruno- Hola, realicé mal el agregado de proyecto granular por eso hice este PR

@matiasmanzanzani
Copy link
Collaborator

Ema, relizaste la prueba con la Custom Allowlist para verificar que tus cambios si funcionan ? En el Readme del repo tenes la explicacion de como usarlo

@omarquisper
Copy link
Contributor Author

Sí ya hice las pruebas y funcionó @matiasmanzanzani

@abruno- abruno- merged commit ac41712 into mercadolibre:master Jan 10, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants