-
Notifications
You must be signed in to change notification settings - Fork 2.7k
Improving selectors of Textbox component #9831
Conversation
@M-ZubairAhmed: Adding the "do-not-merge/release-note-label-needed" label because no release-note block was detected, please follow our release note process to remove it. I understand the commands that are listed here |
@M-ZubairAhmed: Adding the "do-not-merge/release-note-label-needed" label because no release-note block was detected, please follow our release note process to remove it. I understand the commands that are listed here |
8fd16b0
to
284ac35
Compare
Test server with commit |
284ac35
to
caad4c2
Compare
@jwilander Tested autocomplete on the test server https://webapp-perf.test.mattermost.cloud/ autocomplete.mov |
/e2e-test |
Triggering e2e testing with options: |
Successfully triggered e2e testing! |
Awesome, thanks @jgilliam17. Looks good to me. |
components/suggestion/at_mention_provider/at_mention_provider.jsx
Outdated
Show resolved
Hide resolved
Apologies @M-ZubairAhmed I meant to comment, not hit approve |
E2E report looks good, no PR related failures ( master for comparison) |
/cherry-pick release-6.3 |
Cherry pick is scheduled. |
Error trying doing the automated Cherry picking. Please do this manually
|
* improved makeAddLastViewAtToProfiles memoizationg * move makeAddLastViewAtToProfiles & makeGetProfilesInChannel to AtMentionProvider * create getAllDmChannels selector * tests updated test1 * review changes
* improved makeAddLastViewAtToProfiles memoizationg * move makeAddLastViewAtToProfiles & makeGetProfilesInChannel to AtMentionProvider * create getAllDmChannels selector * tests updated test1 * review changes
* improved makeAddLastViewAtToProfiles memoizationg * move makeAddLastViewAtToProfiles & makeGetProfilesInChannel to AtMentionProvider * create getAllDmChannels selector * tests updated test1 * review changes Co-authored-by: Md_ZubairAhmed <[email protected]>
* improved makeAddLastViewAtToProfiles memoizationg * move makeAddLastViewAtToProfiles & makeGetProfilesInChannel to AtMentionProvider * create getAllDmChannels selector * tests updated test1 * review changes
* improved makeAddLastViewAtToProfiles memoizationg * move makeAddLastViewAtToProfiles & makeGetProfilesInChannel to AtMentionProvider * create getAllDmChannels selector * tests updated test1 * review changes Co-authored-by: Md_ZubairAhmed <[email protected]>
…" (#9841) * Improving selectors of Textbox component (#9831) * improved makeAddLastViewAtToProfiles memoizationg * move makeAddLastViewAtToProfiles & makeGetProfilesInChannel to AtMentionProvider * create getAllDmChannels selector * tests updated test1 * review changes * Update snapshots Co-authored-by: Md_ZubairAhmed <[email protected]>
Summary
makeGetProfilesInChannel
&makeAddLastViewAtToProfiles
from Textbox to AtMentionProvider - @jwilandergetAllDmChannels
selectormakeAddLastViewAtToProfiles
selectorTicket Link
Related Pull Requests
N/A
Release Note