-
Notifications
You must be signed in to change notification settings - Fork 453
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 to read the configure #47
base: master
Are you sure you want to change the base?
Conversation
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.
This is a pretty common pattern that we see extension authors use to grab the dialog page for options. I'd just recommend a couple small changes to make it a little clearer for other users.
@madskristensen Why close it? |
Perhaps a mistake. Reopening |
@AlexEyler Can merge? |
Any update? |
The dever want to get the configure but the OptionsPageCS not say how to get the configure.
Add the
static OptionsPagePackageCS EnsurceOptionsPagePackageCs
get the package that can get the configure.