Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(messageStore): adjust getters and tests #12318

Merged
merged 2 commits into from
May 13, 2024
Merged

Conversation

Antreesy
Copy link
Contributor

☑️ Resolves

🖌️ UI Checklist

🖼️ Screenshots / Screencasts

No visual changes, but might fix some visual glitches, while server response hasn't come

🏁 Checklist

  • 🌏 Tested with Chrome, Firefox and Safari or should not be risky to browser differences
  • 🖥️ Tested with Desktop client or should not be risky for it
  • ⛑️ Tests are included or not possible

- jest fails on "rootGetters.getToken is not a function" in further changed tests

Signed-off-by: Maksim Sukharev <[email protected]>
src/services/messagesService.ts Show resolved Hide resolved
src/store/messagesStore.js Outdated Show resolved Hide resolved
src/store/messagesStore.js Outdated Show resolved Hide resolved
@Antreesy Antreesy force-pushed the fix/noid/test-handling branch from 859c9ee to 407aa0a Compare May 10, 2024 17:20
- update messagesService, handle config objects as optional

Signed-off-by: Maksim Sukharev <[email protected]>
@Antreesy Antreesy force-pushed the fix/noid/test-handling branch from 407aa0a to 94ffda6 Compare May 10, 2024 17:27
@Antreesy Antreesy requested a review from DorraJaouad May 13, 2024 10:50
@DorraJaouad DorraJaouad merged commit 4fa57cc into main May 13, 2024
46 checks passed
@DorraJaouad DorraJaouad deleted the fix/noid/test-handling branch May 13, 2024 16:41
@Antreesy
Copy link
Contributor Author

Antreesy commented Jul 3, 2024

/backport to stable29

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

Successfully merging this pull request may close these issues.

2 participants