-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Adding Autocomplete to Stereotype #16025
Adding Autocomplete to Stereotype #16025
Conversation
Hi @MikeAlhayek ,I moved the location of the three class files, otherwise they could not be referenced in other projects. |
@coderabbitai review |
@hyzx86, I've reviewed the changes in the pull request. Here's a summary of the files and their modifications:
Overall, the changes appear to be well-integrated and adhere to the project's standards. If there are specific areas or files you would like a more detailed review on, please let me know! Analysis chainScripts executedThe following scripts were executed for the analysis: Script: #!/bin/bash
# Fetch the list of changed files in the PR
git diff --name-only HEAD~1..HEAD
Length of output: 174 Script: #!/bin/bash
# Fetch the contents of the changed files for review
cat .coderabbit.yaml
cat OrchardCore.sln
cat src/docs/guides/contributing/contributing-code.md
cat src/docs/guides/contributing/reviewing-pull-requests.md
Length of output: 154916 Actions PerformedReview triggered.
|
WalkthroughThe recent updates across various modules in OrchardCore focus on enhancing stereotype management and integration. Key changes include the introduction of Changes
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add CodeRabbit Configration File (
|
@hishamco @MikeAlhayek , This PR is ready for review |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 1
src/OrchardCore.Modules/OrchardCore.ContentTypes/Services/StereotypeService.cs
Outdated
Show resolved
Hide resolved
…eotypeService.cs Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
@hyzx86 I pushed couple cleanup and improvements commits. |
src/OrchardCore.Modules/OrchardCore.ContentTypes/Services/StereotypeService.cs
Outdated
Show resolved
Hide resolved
Seems this merged so fast, that I was waiting for @hyzx86 to request me for a review |
You can always suggest adjustments.😁 |
I know but it would be nice to have this during the review process BTW why the |
Maybe it's automatic? Because the text is the same as the value |
|
||
public Task<IEnumerable<StereotypeDescription>> GetStereotypesAsync() | ||
=> Task.FromResult<IEnumerable<StereotypeDescription>>( | ||
[new StereotypeDescription { Stereotype = "Widget", DisplayName = S["Widget"] }]); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@hishamco here
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Here's the original code.
<datalist id="stereotypeOptions">
<option value="CustomSettings">Custom Settings</option>
<option value="CustomUserSettings">Custom User Settings</option>
<option value="MenuItem">Menu Item</option>
<option value="Widget">Widget</option>
</datalist>
Relate #16005