Skip to content
This repository has been archived by the owner on Apr 4, 2022. It is now read-only.

Missing status when multiple account exist and one account has no messages #33

Closed
tboeckmann opened this issue May 19, 2019 · 1 comment

Comments

@tboeckmann
Copy link
Member

tboeckmann commented May 19, 2019

Describe the bug
When I have 2 or more accounts configured and one or more of them has not yet recieved messages, the field in inbox-messages is blank.

To Reproduce
Steps to reproduce the behavior:

  1. Add 2 accounts (1 with no messages, 1 with at least 1 message)
  2. Go to Inbox and check messages for the account with messages.
  3. Now go to the other account by clicking the address on the left.
  4. Expect the inbox messages to be blank

Expected behavior
It should say: Nothing here yet

Screenshots
2019-05-19_17-45-50

tboeckmann added a commit that referenced this issue May 19, 2019

* fix problem displaying `Nothing here yet` (see issue #33)
tboeckmann added a commit that referenced this issue Jun 16, 2019
* remove dupe sections from readme

* Add TOC

* add documentation to README for server settings
fix naming error (url >> host)

* render message body as courier to support ascii layout

* asciify the message body

* fix problem displaying `Nothing here yet` (see issue #33)

* add ascii invoice used for testing

* add all testnets

* add/ update tests
Add mocks for inboxComponent

* add networks for spec

* Add versioning from package.json (to be semantic versioning)

* skip strange behaviour in test for now

* remove unneeded code in test
skip failing test
tboeckmann added a commit that referenced this issue Jun 16, 2019
* add travis config

* Staging (#45)

* remove dupe sections from readme

* Add TOC

* add documentation to README for server settings
fix naming error (url >> host)

* render message body as courier to support ascii layout

* asciify the message body

* fix problem displaying `Nothing here yet` (see issue #33)

* add ascii invoice used for testing

* add all testnets

* add/ update tests
Add mocks for inboxComponent

* add networks for spec

* Add versioning from package.json (to be semantic versioning)

* skip strange behaviour in test for now

* remove unneeded code in test
skip failing test
tboeckmann added a commit that referenced this issue Jul 5, 2019
* remove dupe sections from readme

* Add TOC

* add documentation to README for server settings
fix naming error (url >> host)

* render message body as courier to support ascii layout

* asciify the message body

* fix problem displaying `Nothing here yet` (see issue #33)

* add ascii invoice used for testing

* add all testnets

* add/ update tests
Add mocks for inboxComponent

* add networks for spec

* Add versioning from package.json (to be semantic versioning)

* skip strange behaviour in test for now

* add coveralls status

* add coveralls

* add coveralls config

* refactor addresses endpoint into its own service

* remove unneeded import

* add protocols service

* add setNetworkList
remove getPublicNetworks method

* fix testing errors
closes #44

* add more coverage

* add some more unit tests

* add tests and document method
tboeckmann added a commit that referenced this issue Jul 22, 2019
* remove dupe sections from readme

* Add TOC

* add documentation to README for server settings
fix naming error (url >> host)

* render message body as courier to support ascii layout

* asciify the message body

* fix problem displaying `Nothing here yet` (see issue #33)

* add ascii invoice used for testing

* add all testnets

* add/ update tests
Add mocks for inboxComponent

* add networks for spec

* Add versioning from package.json (to be semantic versioning)

* skip strange behaviour in test for now

* add coveralls status

* add coveralls

* add coveralls config

* refactor addresses endpoint into its own service

* remove unneeded import

* add protocols service

* add setNetworkList
remove getPublicNetworks method

* fix testing errors
closes #44

* add more coverage

* add some more unit tests

* add tests and document method

* add tests for address pipe
add method documentation

* require args for pipe

* add gitter badge

* add coverage: pipe tests

* remove changeActiveItem method

* inbox tests

* add test to inbox component;
refactor and review inbox component

* add tests
refactor getMails() and move methods to services
tboeckmann added a commit that referenced this issue Jul 24, 2019
* remove dupe sections from readme

* Add TOC

* add documentation to README for server settings
fix naming error (url >> host)

* render message body as courier to support ascii layout

* asciify the message body

* fix problem displaying `Nothing here yet` (see issue #33)

* add ascii invoice used for testing

* add all testnets

* add/ update tests
Add mocks for inboxComponent

* add networks for spec

* Add versioning from package.json (to be semantic versioning)

* skip strange behaviour in test for now

* add coveralls status

* add coveralls

* add coveralls config

* refactor addresses endpoint into its own service

* remove unneeded import

* add protocols service

* add setNetworkList
remove getPublicNetworks method

* fix testing errors
closes #44

* add more coverage

* add some more unit tests

* add tests and document method

* add tests for address pipe
add method documentation

* require args for pipe

* add gitter badge

* add coverage: pipe tests

* remove changeActiveItem method

* inbox tests

* add test to inbox component;
refactor and review inbox component

* add tests
refactor getMails() and move methods to services

* clean up form
add tests
@tboeckmann
Copy link
Member Author

Cannot reproduce. Closing for now.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant