Skip to content

Changed <input> to <select>#26691

Merged
wadepickett merged 2 commits into
dotnet:mainfrom
mrlife:patch-1
Aug 10, 2022
Merged

Changed <input> to <select>#26691
wadepickett merged 2 commits into
dotnet:mainfrom
mrlife:patch-1

Conversation

@mrlife
Copy link
Copy Markdown
Contributor

@mrlife mrlife commented Aug 10, 2022

No description provided.

Copy link
Copy Markdown
Contributor

@wadepickett wadepickett left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. Verified we are using select in the example.

@wadepickett
Copy link
Copy Markdown
Contributor

@mrlife, thanks for taking the time to identify and contribute this fix!

Copy link
Copy Markdown
Contributor

@wadepickett wadepickett left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We have more than one version of this doc in this file. I'll update the others the same way.

Made same correction to 7.0 version of topic.
Copy link
Copy Markdown
Contributor

@wadepickett wadepickett left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Both 6.0 and 7.0 versions fixed. This section did not exist before version 6.0 in the doc.

@wadepickett wadepickett merged commit a2ae14b into dotnet:main Aug 10, 2022
@guardrex
Copy link
Copy Markdown
Collaborator

Thanks guys!!!! Sorry about this. Both Mackinnion and I didn't catch it when I put the original content up over a year ago on Binding select multiple (dotnet/AspNetCore.Docs #22770). Woe! It amazes me how long these 😈 grimilins 😈 go before a sharp 👁️ catches them. Thanks again, @mrlife! 🎷

@mrlife mrlife deleted the patch-1 branch August 16, 2022 12:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants