Skip to content

Conversation

MyPyDavid
Copy link
Member

Description

Related issue: #1413

@MyPyDavid MyPyDavid self-assigned this Sep 30, 2025
Copy link
Member

@jochenklar jochenklar left a 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.

help_text=_('Designates whether this plugin is generally available for projects.')
)

class PluginType(models.TextChoices):
Copy link
Member

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)

Copy link
Member

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.

@MyPyDavid MyPyDavid changed the title Draft: add config app for Plugin model Draft: add rdmo.config app for Plugin model Oct 9, 2025
@MyPyDavid
Copy link
Member Author

MyPyDavid commented Oct 15, 2025

I'm getting some strange DB errors for mysql in ci(and sqlite in local testing).
In my local pytest with sqlite:

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 'rdmo.config' from the installed apps again..
Trying to see it is a bug somewhere in the migrations (0037_remove_attribute.py) but can't really find it so far..

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants