fossasia / open-event-droidgen

Open Event Android App Generator https://github.com/fossasia/open-event-android/raw/apk/sample-apk-fossasia17-development.apk
GNU General Public License v3.0
2.06k stars 828 forks source link

Add appropriate message in notifications fragment when not logged in #2369

Closed ParthivMangukiya closed 6 years ago

ParthivMangukiya commented 6 years ago

Actual Behaviour

Empty view shows "No Notifications" in notification fragment when not logged in.

Expected Behaviour

Empty view should show appropriate message in notification fragment when not logged in.

Screenshots of the issue

20180311_215759

Would you like to work on the issue?

yes

iamareebjamal commented 6 years ago

Showing custom message when you are directly navigated out seems like an overkill

ParthivMangukiya commented 6 years ago

@iamareebjamal I think that navigation flow of opening notification fragment is something odd. If user do not want to login then he have to do two click, one to exit login activity and one for going elsewhere. In my point of view the good flow would be If user click notification in drawer, than he find out he should login to see notification message. There should be Login button at the bottom if he want to login.

But to add login button is out of this issue, so I only updated empty view message.

Please give suggestion, If this flow good than I will create new issue to change navigation flow.

iamareebjamal commented 6 years ago

Yes, please address this issue in a separate PR and also do it for any other fragment which needs the user to be logged in