microsoft / microsoft/WhatTheHack
FastHack 021-IdentityForApps challenge 02 - issue with restricting States to only 3 using StringCollection since the attribute is a built-in attribute
Nobody has claimed this yet.
- Dominant language
- C#
- Stars
- 1.9k
- Forks
- 886
- PR merge metrics
- No merged PRs in 30d
Description
On 021-IdentityForApps challenge 02 there is an issue with the DataType and the form control to use on the data collection for the attribute State/Province.
On challenge 2 of this FastHack there is the request to restrict the list of States/Provinces to only 3 States; and it shows using a StringCollection to achieve a list during sign up.
The challenge reads: "It should be noted that CMC only is licensed to do business in the following states: New York (NY), Ohio (OH) and Pennsylvania (PA) (values between parentheses represent the values that should be stored in the directory)."
This particular attribute (State/Province) is a Built-In attribute of type String and you cannot modify its DataType to be a StringCollection.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with 021-IdentityForApps/Coach/02-susi.md and compare the State/Province instructions with the built-in attribute's String type. Clarify how the challenge should represent the three permitted states without treating State/Province as a StringCollection, then verify the revised instructions are consistent with the stated directory values.
Written by the indexing model from the issue text.
Assessment
- Domain
- documentation
- Issue type
- Documentation
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100