-
Notifications
You must be signed in to change notification settings - Fork 52
Draft: add rdmo.config
app for Plugin model
#1436
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
base: 2.3.3
Are you sure you want to change the base?
Conversation
Signed-off-by: David Wallace <[email protected]>
Signed-off-by: David Wallace <[email protected]>
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.
Great! Lets discuss in the meeting.
rdmo/config/models.py
Outdated
help_text=_('Designates whether this plugin is generally available for projects.') | ||
) | ||
|
||
class PluginType(models.TextChoices): |
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.
I think we can do without this, the Plugin(-class) should know this.
verbose_name=_("Python path"), | ||
) | ||
plugin_type = models.CharField(max_length=32, choices=PluginType.choices) | ||
|
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.
But what we need to add is plugin_settings
which is used to set the settings
for this plugin.
rdmo.config
app for Plugin model
Signed-off-by: David Wallace <[email protected]>
Signed-off-by: David Wallace <[email protected]>
I'm getting some strange DB errors for mysql in ci(and sqlite in local testing). django.db.utils.OperationalError: foreign key mismatch - "questions_question" referencing "domain_attribute" From the ci logs: # at
query = b'ALTER TABLE `domain_attribute` DROP COLUMN `attributeentity_ptr_id`' django.db.utils.OperationalError:
(1829, "Cannot drop column 'attributeentity_ptr_id':
needed in a foreign key constraint 'questions_questionse_attribute_id_30ee3ea9_fk_domain_at' of table 'questions_questionset'") The problem can be resolved by removing the app |
Signed-off-by: David Wallace <[email protected]>
Signed-off-by: David Wallace <[email protected]>
Description
Related issue: #1413