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

[Feature] .EnableBurkusMvvmDebugMessages() #17

Open
BurkusCat opened this issue Oct 4, 2023 · 0 comments
Open

[Feature] .EnableBurkusMvvmDebugMessages() #17

BurkusCat opened this issue Oct 4, 2023 · 0 comments
Labels
enhancement New feature or request

Comments

@BurkusCat
Copy link
Owner

Is your feature request related to a problem? Please describe.
It would be good if the library provided a way to expose extra debug logging from errors and warnings that occurred within the library.

Describe the solution you'd like
Example: It would be good if the library provided a warning if you tried to automatically set a property (#16) but the navigation parameter key did not exist.

Describe alternatives you've considered
N/A

Proposed APIs
If possible, try to propose how the API might look. For example:

// BurkusMvvmBuilderExtensions
.EnableBurkusMvvmDebugMessages()

Additional context
Should the code in this method be surrounded with

#IF DEBUG

#ENDIF

... maybe it would be better to have this code as an example around the builder method usage in the documentation and leave it up to user choice if they wanted to include this in a RELEASE build.

@BurkusCat BurkusCat added the enhancement New feature or request label Oct 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant