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

Auto fill user info if logged in #512

Closed
aparveen opened this issue Sep 8, 2021 · 9 comments
Closed

Auto fill user info if logged in #512

aparveen opened this issue Sep 8, 2021 · 9 comments

Comments

@aparveen
Copy link
Contributor

aparveen commented Sep 8, 2021

• If a user is logged in, we can query information from the user like first name, last name, and e-mail.
• This enhancement would allow us to configure fields in the report form and comment form to auto populate with this information from the user.
• Note in the screenshot, Phone Number is not a property on the portal user so it can’t be autopopulated.
image

@aparveen
Copy link
Contributor Author

aparveen commented Sep 8, 2021

@CTLocalGovTeam could you please assess this for 9.4 release? Thanks

@ashutoshverma-ct
Copy link

ashutoshverma-ct commented Sep 22, 2021

@aparveen

We understood the requirements of this enhancement but we need some more information:

If a user is logged in, we can query information from the user like first name, last name, and e-mail.

Q1. For a logged-in user we get several values (attached as a text file - Sample user info.txt), do we have to honor only first name, last name, and e-mail?

• This enhancement would allow us to configure fields in the report form and comment form to auto populate with this information from the user.

Q2. To auto-populate, the fields with the logged-in user, do we have to match the exact field names with the portal user properties? For instance, we get "fullName" from the portal user as a property so its value should be auto-populate to the field in the report with the same name and not with the field "name"

Q3. Should we match with the displayed field name (alias) of the report only or with the actual field name too?

@skrishn
Copy link
Contributor

skrishn commented Sep 29, 2021

@ashutoshverma-ct
Q1 - Yes. you're correct.
Q2 - We expose this in the config panel so that users choose how the fields are matched.
Q3 - Do we expose alias and actual field name in the config for any other parameter in this app? Please confirm what we do. I'd suggest we use the same pattern we did for other parameters

Let me know if you have additional questions.

cc @aparveen

@ashutoshverma-ct
Copy link

@aparveen @skrishn

As discussed in the meeting, the workflow will be as follows:

  • In the configuration, we will provide 3 textboxes to specify field name to auto-populate user info (i.e. first name, last name and email id)
    • User should be able to specify only 1 field name
  • If the specified field name is not found in the feature layer then the value will not be auto-populate
  • Data type validation and length truncation will be handled
  • All the auto-populated fields will remain editable for the users to edit the value

ashutoshverma-ct pushed a commit to ashutoshverma-ct/crowdsource-reporter that referenced this issue Nov 9, 2021
### Implemented following gitHub tickets:

 - Esri#510 (Configuration changes)
 - Esri#512
 - Esri#513
sandhyakurumanale-ct added a commit to sandhyakurumanale-ct/crowdsource-reporter that referenced this issue Nov 9, 2021
### Implemented following  gitHub tickets:

 - Esri#510
 - Esri#512
 - Esri#513
 - String changes
sumitzarkar added a commit to sumitzarkar/crowdsource-reporter that referenced this issue Nov 19, 2021
### Implemented following git hub tickets
 - Esri#510
 - Esri#512
 - Esri#513
 - Esri#516
@aparveen
Copy link
Contributor Author

Test case? @skrishn

@aparveen
Copy link
Contributor Author

aparveen commented Nov 23, 2021

@CTLocalGovTeam What will be the level of effort to apply this to comment table? In our solution schema, related comment table and feature layer has the same field names for these fields.
It will help simplify editing experience for our users.
Could you please evaluate this and see if it can be done in 9.4?
image

@sumitzarkar
Copy link
Contributor

@aparveen

We can add this functionality to the comment forms as well, this would need low level of efforts.
We have implemented this functionality today and currently we are testing it. We will deliver this once our testing is completed.

@aparveen
Copy link
Contributor Author

@sumitzarkar Thanks so much for your quick action on this, appreciate it.

sandhyakurumanale-ct added a commit to sandhyakurumanale-ct/crowdsource-reporter that referenced this issue Nov 25, 2021
### Implemented following ticket :

 - Esri#512
@skrishn
Copy link
Contributor

skrishn commented Dec 1, 2021

verified in qa

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

No branches or pull requests

5 participants