[Team Management 2.0] Do we need auto complete for adding existing topcoder users

[vikasrohit]

Currently after new team management section, we are missing the auto complete feature for searching existing topcoder user while adding a user to the team. It might be an intentional feature, however, creating the issue to track the discussion.

fyi @acshields @vic-topcoder @RishiRajSahu

[vikasrohit]

@acshields as per our last discussion, we need this feature again so we are including this our bug bash.

[vikasrohit]

@maxceem one important thing to keep in mind for this feature is that we should be able to invite the member by handle even if there is no result returned for that by the api.

@acshields @vic-topcoder could you please confirm that this is an expected behaviour?

[maxceem]

IMPORTANT
/~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
I've just found that recently we already added a component for users autocomplete. Most likely it could be easily reused with some configuration for our case which makes the task much simpler.
Please, have look at this forum post for details: https://apps.topcoder.com/forums/?module=Thread&threadID=931421&mc=1#2318673
/~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

To sum up:

• this featuer should work in both dialogs Project Team and Topcoder Team. We can see them by using user pshah_manager and opeining this project https://connect.topcoder-dev.com/projects/7839 by clicking the Manage link on the bottom of the left sidebar.

  

• we previously had autocomplete working in the previous implementation of team management feature. To check it out can use commit a9b60bc8f2fd845d3eff82b860033f192e824ba3. See demo https://monosnap.com/file/zmx4r3Gp4ENHgrkmG59SJmqJOXZlDh Previous implementation could be used for reference.

• now the suggestion list should be shown under the field, not on above the field like before

• we should remove the requirement to start handles from @. In both dialogs, we should be able to enter handles without @. So validation should also let us do it. And we should send to the server correct data which server expects. It would lead to the fact that we would show suggestions even when we are typing email address, but it's fine.

• update placeholder texts in Topcoder team, so it doesn't suggest inputting handle with @

• even if there are no results returned by autocomplete we should be able to type handle ourself and invite user (unlike in previous implementention)

• we should support navigating by keyboard (didn't have before):

  • press down/up should select next/previous suggested handle
  • pressing Enter should complete the typing handle with the currently selected handle

• note, that we can type several handles in the field and in Project team dialog we can type both handles and email.

[vikasrohit]

• now we should start typing user handle with @ to make autocomplete work. As we also can type email which don't start from @ and they shouldn't have autocomlete

@maxceem we have a new requirement from the team that @ is causing confusion to the product users, so we need to get rid of it. So, could we also get rid of @ now along with this task? I know the side effect would be that it would make server calls even when user is typing emails, but I guess that does not harm, user might find existing user for that email in this case.

[maxceem]

@vikasrohit got it. Should we let users enter handles both ways with or without @? @handle and handle?

[vikasrohit]

I think drop the support for @, using both might be more confusing. We need to update the help text of the input as well after this.

[cagdas001]

@maxceem I'm using the feature/team-management-autocomplete branch, as indicated. But I can't open any project (including the mentioned one) I'm getting an 404 page. Attached is the what I see

This is happening in both local and remote build.

Note: I don't know which branch you're using at https://connect.topcoder-dev.com, but as I said, same issue appears also there.

I tried on my mobile device's browser as well (which doesn't have /etc/hosts entry), so most likely this is a valid issue and I'm not the only one who is having this issue.

Edit

It seems this issue is related to API. See the console log

I think this will be resolved by itself, once its fixed in the API repo. But its blocking this issue for now

[maxceem]

Yeah, @cagdas001 looks like something got broken with project service. I'll try to get it fixed and updated to you asap.