glucosio-android icon indicating copy to clipboard operation
glucosio-android copied to clipboard

Use Support navigation drawer instead of third party

Open emartynov opened this issue 7 years ago • 9 comments

I've removed date/time pickers and FAB. But removing drawer requires a bit more time.

emartynov avatar Jan 14 '18 07:01 emartynov

Drawer library isn't deprecated and is well supported. We can keep it IMHO.

paolorotolo avatar Jan 14 '18 19:01 paolorotolo

Both variants seem fine to me. I did the support lib drawer implementation for Nextcloud so that is doable while the lib offers a lot of on top functionality. So both have their upsides.

AndyScherzinger avatar Jan 14 '18 19:01 AndyScherzinger

Up to you. As for me, less third-party dependencies is better.

emartynov avatar Jan 14 '18 19:01 emartynov

I’d leave the decision to @paolorotolo

AndyScherzinger avatar Jan 14 '18 20:01 AndyScherzinger

I've asked Mike about it https://github.com/mikepenz/MaterialDrawer/pull/2171

emartynov avatar Jan 14 '18 20:01 emartynov

Yes, I like the idea to reduce the amount of third-party libraries in the app but I also don't want to reinvent the wheel. I've seen Mike answer and for now our implementation of the drawer is pretty standard and could be replaced by the Support Library without issues.

On another side, we might need to introduce login very soon (to enable @chrispecoraro's API) and add Account support to the drawer, and this is easily achievable with Mike Penz's library.

paolorotolo avatar Jan 14 '18 20:01 paolorotolo

I'm not sure about login for the user. It doesn't give much additional functionality and only will be added by developers to achieve unique ID for uploaded data. That might be resolved by just GUID generation.

Unless we have something on the roadmap that will also use login.

emartynov avatar Jan 14 '18 20:01 emartynov

No, as @emartynov said we only need login to get an ID for our API. @bkerensa can say more about our vision, one of the key principles of our project is to keep users in complete control of their data, for this I wouldn't consider for now the option to upload data to third-party servers.

Unless, if we really want to explore the possibility of sync between devices, we can add login to a personal NextCloud server or something like that. But again, we need to discuss this properly and we're going a little OT here.

paolorotolo avatar Jan 14 '18 21:01 paolorotolo

@emartynov @paolorotolo guys, could you update projects kanban? It looks old and has some issues are closed.

piotrek1543 avatar Jun 02 '18 19:06 piotrek1543