rokwire / rokwire-building-blocks-api

Source code repository of the Rokwire App Config, Authentication, Events, Profile, and Logging Building Blocks, and Authentication Middleware Library.
Apache License 2.0
3 stars 4 forks source link

1116 catalog texts and placeholders #1121

Closed minump closed 1 year ago

minump commented 1 year ago

Description

Changed form field and placeholder wordings for contribute form.

Fixes #1116

Review Time Estimate

Please give your idea of how soon this pull request needs to be reviewed by selecting one of the options below. This can be based on the criticality of the issue at hand and/or other relevant factors.

Type of changes

Please select a relevant option:

Checklist:

Please select all applicable options:


bingzhang commented 1 year ago

I did not check the text with google Docs. but the code looks good.

minump commented 1 year ago

some questions : "Select the Contribution Type and Add Details". --- this was "Select the Contribution Type" before. Since the label is for "Add Talent" and "Add Capability" buttons, do we need to change the label? The one lien description for "Add a Capability" label has been delete. Is the description for "Add a Talent" label is also deleted? As of now, in the google docs it is "Add a Talent (a User Interface Component that Provides a Specific Functionality)"

sandeep-ps commented 1 year ago

"Select the Contribution Type and Add Details". --- this was "Select the Contribution Type" before. Since the label is for "Add Talent" and "Add Capability" buttons, do we need to change the label? The one lien description for "Add a Capability" label has been delete. Is the description for "Add a Talent" label is also deleted? As of now, in the google docs it is "Add a Talent (a User Interface Component that Provides a Specific Functionality)"

I thought that when a user clicks that button, it's not just like selecting an option; a user needs to add more details. It's not very important, I think. I have updated the suggestions in the Google doc to fix the inconsistencies. Please see. Thanks for catching that.

minump commented 1 year ago

"Select the Contribution Type and Add Details". --- this was "Select the Contribution Type" before. Since the label is for "Add Talent" and "Add Capability" buttons, do we need to change the label? The one lien description for "Add a Capability" label has been delete. Is the description for "Add a Talent" label is also deleted? As of now, in the google docs it is "Add a Talent (a User Interface Component that Provides a Specific Functionality)"

I thought that when a user clicks that button, it's not just like selecting an option; a user needs to add more details. It's not very important, I think. I have updated the suggestions in the Google doc to fix the inconsistencies. Please see. Thanks for catching that.

Updated labels and placeholders. Ready for review.

minump commented 1 year ago

Merging to develop