Automattic/ScreenObject

Add method to dismiss Simulator request to save password

Aperta

#13 aperta il 26 apr 2023

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)Swift (2 fork)auto 404
good first issue

Metriche repository

Star
 (6 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

See logic duplicated between https://github.com/wordpress-mobile/WordPress-iOS/pull/20583 and https://github.com/woocommerce/woocommerce-ios/pull/9555.

Despite opening this issue, I'm a bit on the fence on this one. It would be handy to hide the logic away here, but at the same time it's really just a couple of lines and it might be easier to keep duplicating it across apps and let each consumer adapt its own version.

Guida contributor

Direzione di ricerca
Esamina il codice duplicato nelle PR referenziate e decidi se centralizzare la logica di chiusura della richiesta di salvataggio password in ScreenObject o mantenerla per app. Quindi implementa il metodo in ScreenObject.
Tech stack
swift
Dominio
mobile
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
1
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
SwiftiOS development
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60