Code review comment for lp:~cellsoftware/telegram-app/1599030

Revision history for this message
Jin (jindallo) wrote :

Hello Ash,

After .cache and .cache/QML/App/com.ubuntu.telegram* deleted,
I re-installed the app. on my phone and still has below findings:
    1. the app. is closed
    - leads user to the proper dialog window after notification pressed (OK)
    - then click back key on dialog window returns to main view but the chat is focused
      (this will be a regression of bug: 1576027)
    2. the app. is in the background as main view (AccountListPage)
    - No leads user to the dialog window, it keeps the page as main view (NG)
    3. the app. is in the background as the same dialog window (AccountDialogPage)
    - leads user to the proper dialog window after notification pressed (OK)
    4. the app. is in the background as the different dialog window (AccountDialogPage)
    - it shows a "Please select chat" broken page (NG)

Above "same" means in the dialog you are talking to the person who send you the messages,
and the "different" means the person who send you the messages is different with the one you are talking to.

review: Needs Fixing

« Back to merge proposal