-
Notifications
You must be signed in to change notification settings - Fork 91
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
Unit settings #1640
Comments
Could move this to #1604 or a different future epic depending on how it makes sense to group sidebar/unit features |
@jmakowski1123 @sdaitzman Can we reduce the scope here for the MVP / initial epic 12? Maybe make it read-only or remove it entirely? |
@bradenmacdonald I think we could, but it interacts a bit with #1602. I think it would be a strange product experience to be able to import a unit into a library, but not be able to change the settings back to defaults. That being said, I think it's a bit lower-impact for units than it would be for section/subsection, since the only settings are visibility and enable/disable discussions. |
@sdaitzman In that case, the simpler story could be "reset to defaults when importing units into a library". But I'm fine with implementing this as is. It's not a big story. |
Screenshots
Spec
In Scope
Not in scope
The text was updated successfully, but these errors were encountered: