-
Notifications
You must be signed in to change notification settings - Fork 2
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
Feature/cu 34ga47h - News #95
Conversation
- Added new functions in MainApi - Changed function to get date of days before an other date - If a tag has no image, take the image of its first article - Yellow background if there is no image - Added favourite tag feature - Added highligh article - Load new articles as you scroll - Code refactoring
- Added a NewsManager component that downloads and processes all the data
- minor changes and refactoring
- Loading indicator while fetching articles - Event to update preferences instead of callback - Code refactoring
- from RN Swith to third party package customizable Switch - capitalize tag name when displayed - refactoring
AH, altra cosa. Ho aggiunto una dependency per fare il toggle switch perché il Native Switch su android era diverso, più stile android ma davvero orrendo. E creare un componente da zero mi sembrara un po' un overkill. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok scusa se ho fatto la review solo adesso, ci sono stati problemi con il backend che sono stati risolti solo ieri e non riuscivo per un cazzo a far funzionare gli articoli lol, comunque ottimo lavoro! veramente monumentale anche questo, molto preciso e pulito.
mi piace un sacco l'uso del DeviceEventEmitter per il problema della chiusura del menu, anche se è tecnicamente deprecato e non è per un cazzo il suo intended use (volendo si potrebbe molto più banalmente creare un EventEmitter custom, così si potrebbe anche tipizzare meglio quali eventi esistono senza usare costanti in giro, vedi per esempio HttpClient.ts), mi piace meno usato per settare o meno i preferiti ahah, quella cosa sarebbe meglio modificarla
probabilmente ora dovrebbe essere fixato credo? la sezione "altre categorie" ora è gestita come qualsiasi altra pagina nella navigazione |
@toto04 lasciamo stare il bug che ho menzionato per questa pr, ne discuteremo in un issue a parte e faremo branch e pr a parte per quando sarà |
anche #124 non è importante per questa branch/pr, lo faremo in una branch/pr a parte |
perfetto, ce ne occuperemo nel prossimo sprint in tal caso 👌 |
Status task aggiornato: CU-34ga47h - completed |
Mi manca solo da capire come fare il tasto "Altre categorie" da un punto di vista di design, però intanto faccio il commit.
Il passaggio dalla schermata "categorie preferite" alla schermata "altre categorie" l'ho fatto come modale e non come navigation perchè così si può sempre chiudere la tendina scrollando in basso, e poi perchè ho notato che passare parametri quando torni indietro nella navigazione non è proprio così semplice ahahah. Però se secondo te è meglio farlo come navigation, ci guardo.
Poi dovrei anche aver sistemato la cosa che cliccando il tasto home la tendina si chiude, usando i DeviceEvent.