openMF / mifos-mobile

Repository for the Mifos Mobile Banking App for clients
https://openmf.github.io/mobileapps.github.io/
Mozilla Public License 2.0
270 stars 674 forks source link

Update App screens shots in readme #2520

Closed therajanmaurya closed 5 months ago

therajanmaurya commented 6 months ago

Summary:

Summarize your issue in one sentence (what goes wrong, what did you expect to happen).

Steps to reproduce:

How can we reproduce the issue?

Expected behaviour:

What did you expect the app to do?

Observed behaviour:

What did you see instead? Describe your issue in detail here.

Device and Android version:

What make and model device (e.g., Samsung Galaxy S3) did you encounter this on? What Android version (e.g., Android 4.0 Ice Cream Sandwich or Android 6.0 Marshmallow) are you running? Is it the stock version from the manufacturer or a custom ROM?

Screenshots:

Can be created by pressing the Volume Down and Power Button at the same time on Android 4.0 and higher.

Akshay2004-701 commented 6 months ago

I would like to do this task, can I be assigned with it?

kmanikanta335 commented 6 months ago

@therajanmaurya can i work on this issue ?

AvneetSingh2001 commented 6 months ago

Hey @Akshay2004-701, as you commented first, so please directly make a PR and send the PR link in the Slack Channel. Thanks!

Vikashgathala commented 5 months ago

@Akshay2004-701 Can I go with this if you're okay with it?

Akshay2004-701 commented 5 months ago

ok, sure

Vikashgathala commented 5 months ago

Thank you @Akshay2004-701 . @PratyushSingh07 Please assign this to me.

PratyushSingh07 commented 5 months ago

Go ahead @Vikashgathala

Vikashgathala commented 5 months ago

@PratyushSingh07 Do I need to add the screenshots of the same screens and in the same order as previous ones? I am unable to add beneficiaries so same activities with new UI isnt possible right now.

PratyushSingh07 commented 5 months ago

In that case add UI of those screens that are visible

Vikashgathala commented 5 months ago

@PratyushSingh07 Have a look at this. LGTM.

PratyushSingh07 commented 5 months ago

Instead of loan account details we can show Help section. Moreover, I would prefer having something similar to the one used in Android client. Let me know if you can replicate that

Vikashgathala commented 5 months ago

@PratyushSingh07 I'll do this by evening.

PratyushSingh07 commented 5 months ago

Great, thanks

Vikashgathala commented 5 months ago

@PratyushSingh07 Done.

PratyushSingh07 commented 5 months ago

Lgtm @Vikashgathala . Can you just replace the current help section screenshot with the one in which we can have some of the answers as well. I mean can you expand a few of them . It would look better imo. Let me know if you think otherwise

Vikashgathala commented 5 months ago

There is no UI separation between the help section's question and solution after expanding. I tried that but it looks like a single paragraph. This looks a little better and we have any ui improvements in help section in future, it would be better to update it in this. also, I have squashed the commits and its ready from my side until you find it mandatory to update the screenshots as you are recommending. Let me know what you think. @PratyushSingh07

PratyushSingh07 commented 5 months ago

Lgtm , we can go ahead with this. Please open a PR. It seems that you have just pushed the changes to your fork and haven't actually opened the PR