anitab-org / mentorship-android

Mentorship System is an application that matches women in tech to mentor each other, on career development, through 1:1 relations during a certain period of time. This is the Android application of this project.
https://github.com/anitab-org/mentorship-android/raw/apk/app-debug.apk
GNU General Public License v3.0
164 stars 379 forks source link

Storing the data locally without going online. #787

Open garimasingh128 opened 4 years ago

garimasingh128 commented 4 years ago

Description

As an user of the app, sometimes I may need to check my profile or requests but network connection may not be available. So my profile data, requests and such other information should be stored locally in my phone. For example in Whatsapp, you can check the chats even if you're offline. In the same way, I should be able to check my stats on the app without going online. We can use room database for storing the data locally.

Acceptance Criteria

Update [Required]

Definition of Done

Estimation

2-4 hours depending on fluency with the backend and database.

niyati-n commented 4 years ago

Can you assign this issue to me for OSH ?

aditmehta9 commented 4 years ago

Can you assign this issue to me for OSH ?

Yes of course. OSH starts on 1st September and I can't officially assign you just yet. Your PR will be considered for OSH only if you're assigned after 1st. Before that, if I assign you it'll be a regular contribution and please tag project managers to claim issue.

prayutsu commented 4 years ago

I could also work on it, so kindly assign it to me, if in case @blackcrabb is unable to take up this issue.

niyati-n commented 4 years ago

@garimasingh128 i would like to work on this issue

aditmehta9 commented 4 years ago

Can you assign this issue to me for OSH ?

Assigned.

vj-codes commented 4 years ago

@aditmehta9 @blackcrabb is already assigned #795 first so unassigning for now. @blackcrabb Please work on one issue at a time and once done claim another .Thanks!

aditmehta9 commented 4 years ago

@aditmehta9 @blackcrabb is already assigned #795 first so unassigning for now. @blackcrabb Please work on one issue at a time and once done claim another .Thanks!

Thanks! I didn't noticied.

sanchi0204 commented 4 years ago

I could also work on it, so kindly assign it to me, if in case @blackcrabb is unable to take up this issue.

Assigning you @prayutsu

anna4j commented 4 years ago

I could also work on it, so kindly assign it to me, if in case @blackcrabb is unable to take up this issue.

Assigning you @prayutsu

he's already assigned to another issue

CodeChamp-SS commented 4 years ago

@garimasingh128 @aditmehta9 @anna4j i would like to take up this issue :-) Can u pls tell what all data needs to be saved for offline viewing ?

garimasingh128 commented 4 years ago

@anna4j you wanna add something?

CodeChamp-SS commented 4 years ago
  • His own profile details
  • Requests

@anna4j you wanna add something?

@garimasingh128 ok, so shall i start working on the issue ?

anna4j commented 4 years ago
  • His own profile details
  • Requests

@anna4j you wanna add something?

Sorry I missed it earlier. User should also be able to see their tasks offline

vj-codes commented 4 years ago

@anna4j @garimasingh128 hey idk how but this issue showed up in my notifications .There was already a similar issue #714 and a PR #718 was created but after very long discussion both PR and issue was put up on hold by Isabel. Just letting you guys know, the status of this one (approved/hold) is upto you .Thanks!

garimasingh128 commented 4 years ago

For now, I think let's wait how @CodeChamp-SS implements the process and have a look at his PR. Then we can see whether it should be merged or not. What's say @vj-codes ?

CodeChamp-SS commented 4 years ago

shall i make use of coroutines in room ? (i think it would be better if it's used)

CodeChamp-SS commented 4 years ago

@garimasingh128 as i am adding room dependencies to the app, so shall i also update other dependencies version and gradle version for the app ?

garimasingh128 commented 4 years ago

@CodeChamp-SS Yeah you can but make sure nothing else breaks and keep testing the other components as well.

CodeChamp-SS commented 4 years ago

@CodeChamp-SS Yeah you can but make sure nothing else breaks and keep testing the other components as well. ok

CodeChamp-SS commented 4 years ago

shall i make use of coroutines in room ? (i think it would be better if it's used)

@garimasingh128 nd what abt this ?

garimasingh128 commented 4 years ago

I have never implemented Coroutines myself so if you have used them earlier and pretty confident about it, you can go ahead. I will do a little research about them and get back to you in a day or two.

CodeChamp-SS commented 4 years ago

I have never implemented Coroutines myself so if you have used them earlier and pretty confident about it, you can go ahead. I will do a little research about them and get back to you in a day or two.

ya i've used it earlier but you can take your time and tell me tell if i should use it, till then i'll try to implement coroutines with room in the app

CodeChamp-SS commented 4 years ago

issue @garimasingh128 @anna4j fab for adding tasks is hidden under the bottom nav view, is this a valid issue ? If yes, then i can also fix this issue

anna4j commented 4 years ago

@garimasingh128 should we let him open an issue for this and then work on it? Or work on it under this issue itself

vj-codes commented 4 years ago

@CodeChamp-SS try scrolling down the page and then the button is visible There was a issue #744 about the and PR #745 which is merged now cc: @anna4j

CodeChamp-SS commented 4 years ago

@CodeChamp-SS try scrolling down the page and then the button is visible There was a issue #744 about the and PR #745 which is merged now cc: @anna4j

no @vj-codes not visible to me

CodeChamp-SS commented 4 years ago

@anna4j @garimasingh128 can either of you please confirm whether you can see the fab or not ?

vj-codes commented 4 years ago

@CodeChamp-SS which device? This bug is strange, just checked now on mine and it's visible Screenshot_2020-09-18-19-03-57-531_org.systers.mentorship.jpg

CodeChamp-SS commented 4 years ago

@CodeChamp-SS which device? This bug is strange, just checked now on mine and it's visible Screenshot_2020-09-18-19-03-57-531_org.systers.mentorship.jpg

i just checked out the pr, the changes made their in the fragment_mentorship_tasks.xml are not reflecting in my cloned repo

CodeChamp-SS commented 4 years ago

if the issue has already been fixed, then it's fine

vj-codes commented 4 years ago

@CodeChamp-SS your fork is 4 commits behind the develop branch that inclues the task button commit too , if you sync your fork it'll work ! Or uninstall current apk and download apk from this main repo and not your clone on your device

CodeChamp-SS commented 4 years ago

@CodeChamp-SS your fork is 4 commits behind the develop branch that inclues the task button commit too , if you sync your fork it'll work ! Or uninstall current apk and download apk from this main repo and not your clone on your device

ya i'm syncing it now

CodeChamp-SS commented 4 years ago

@anna4j @garimasingh128 i've implemented saving profile for offline view, but while implementing saving tasks and requests i realised that a huge amount of code base has to be changes in order to achieve so. So what shall i do ? Shall i make a pr with only feature of offline viewing of profile or what ?

garimasingh128 commented 4 years ago

I think too much change in code would be too much to do for a single issue. We can store profile data offline for now and create separate issues for saving tasks and requests. What's say @anna4j @vj-codes ?

vj-codes commented 4 years ago

@garimasingh128 @CodeChamp-SS agreed for now just profile view is good , do update the issue accordingly though:)

CodeChamp-SS commented 4 years ago

i've implemented the offline viewing of data in the profile fragment, i.e, the profile details. Is it fine ?

CodeChamp-SS commented 4 years ago

i've implemented the offline viewing of data in the profile fragment, i.e, the profile details. Is it fine ?

shall i make the pr with these changes ?

garimasingh128 commented 4 years ago

Ya go ahead

kartikeysaran commented 3 years ago

If this issue is still open , Can i work on this ?

epicadk commented 3 years ago

If this issue is still open , Can i work on this ?

Sorry a pr has already been created. Removing status available