-
Notifications
You must be signed in to change notification settings - Fork 34
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
All Option Descriptions Should Be Extensible #27
Comments
Committed Magritte-Model-SeanDeNigris.466 Now MultipleOptionDescriptions can be #extensible, too! Implementation note: #initializer had to be pushed up, since we need to supply a way to create the new instance. This pushes us a bit closer to my suspicion that #classes for reference descriptions should be #factories, which would probably mean a name-change for #initializer |
seandenigris
pushed a commit
to seandenigris/Magritte
that referenced
this issue
Jul 27, 2016
…: All Option Descriptions Should Be Extensible; now MultipleOptionDescriptions can be #extensible, too! Implementation note: #initializer had to be pushed up, since we need to supply a way to create the new instance. This pushes us a bit closer to my suspicion that #classes for reference descriptions should be #factories, which would probably mean a name-change for #initializer
noha
pushed a commit
to zweidenker/magritte
that referenced
this issue
Aug 8, 2018
…d Be Extensible; now MultipleOptionDescriptions can be #extensible, too! Implementation note: #initializer had to be pushed up, since we need to supply a way to create the new instance. This pushes us a bit closer to my suspicion that #classes for reference descriptions should be #factories, which would probably mean a name-change for #initializer
Fixed by 039957b |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Right now, only Single Option Descriptions allow you to add a new object not included in the initial options, but there doesn't seem to be anything specific to the Single-Option domain.
The text was updated successfully, but these errors were encountered: