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

Add docs for ThemeSelectorResult #11147

Merged
merged 4 commits into from
Mar 13, 2024
Merged

Add docs for ThemeSelectorResult #11147

merged 4 commits into from
Mar 13, 2024

Conversation

hishamco
Copy link
Member

@hishamco hishamco commented Feb 7, 2022

This issue here is to avoid the expectation of ThemeName many of the dev may expect the theme name is used in ThemeSelectorResult while the identifier is used instead. @agriffard if we need add remark on the docs please let me know where should we add it

@hishamco hishamco requested a review from agriffard February 7, 2022 22:35
@hishamco hishamco closed this Feb 7, 2022
@hishamco hishamco reopened this Feb 7, 2022
@hishamco
Copy link
Member Author

hishamco commented Feb 7, 2022

What's the wrong with https://nodejs.org/dist/v15.14.0?!!

@hishamco hishamco requested a review from ns8482e January 23, 2024 11:10
@hishamco hishamco requested a review from Skrypt February 23, 2024 23:23
@Piedone Piedone merged commit dc6dab0 into main Mar 13, 2024
5 checks passed
@Piedone Piedone deleted the hishamco/theme-selector-result branch March 13, 2024 17:25
urbanit pushed a commit to urbanit/OrchardCore that referenced this pull request Mar 18, 2024
Co-authored-by: Dean Marcussen <dean.marcussen@gmail.com>
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.

5 participants