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.

Sign up for GitHub

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

Jump to bottom

Provide OnTopicOptions class for AddTopicSupport() #98

Open
JeremyCaney opened this issue Dec 6, 2021 · 1 comment
Open

Provide OnTopicOptions class for AddTopicSupport() #98

JeremyCaney opened this issue Dec 6, 2021 · 1 comment
Labels
Area: Web Relates to the `AspNetCore` or other web-related functionality. Priority: 3 Severity 0: Nice to have Status 0: Discussion Needs further evaluation of requirements and prioritization. Type: Improvement Improves the functionality or interface of an existing feature.

Comments

@JeremyCaney
Copy link
Member

JeremyCaney commented Dec 6, 2021 •
edited

The UseMvc() extensions accepts an optional MvcOptions object for configuration. We may optionally be able to centralize the most common OnTopic configuration through this (example).
@JeremyCaney JeremyCaney added Area: Web Relates to the `AspNetCore` or other web-related functionality. Severity 0: Nice to have Priority: 3 Type: Improvement Improves the functionality or interface of an existing feature. Status 2: Scheduled Planned for an upcoming release. labels Dec 6, 2021
@JeremyCaney JeremyCaney added this to the OnTopic 5.2.0 milestone Dec 6, 2021
@JeremyCaney JeremyCaney self-assigned this Dec 6, 2021
@JeremyCaney JeremyCaney added Status 0: Discussion Needs further evaluation of requirements and prioritization. and removed Status 2: Scheduled Planned for an upcoming release. labels Dec 6, 2021
@JeremyCaney JeremyCaney removed their assignment Dec 6, 2021
@JeremyCaney JeremyCaney removed this from the OnTopic 5.2.0 milestone Dec 6, 2021
@JeremyCaney
Copy link
Member Author

This potentially overlaps with #13, which proposes allowing the OnTopic library to be configured via OnTopic. As such, we should take care to focus on the features of OnTopic that must be determined in code (such as elements which require delegates or type references) or which need to be known upfront (such as the connection string), as opposed to basic values that can easily be assigned once the Root:Configuration section has been successfully loaded.

@JeremyCaney JeremyCaney changed the title Provide OnTopicOptions class for MVC Provide OnTopicOptions class for AddTopicSupport() Jan 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Area: Web Relates to the `AspNetCore` or other web-related functionality. Priority: 3 Severity 0: Nice to have Status 0: Discussion Needs further evaluation of requirements and prioritization. Type: Improvement Improves the functionality or interface of an existing feature.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@JeremyCaney