Skip to content

Fix related name conflicts and allow sqlite migrations#270

Merged
2-Coatl merged 6 commits intodevelopfrom
feature/explain-runserver-command-behavior-15-25-19
Nov 18, 2025
Merged

Fix related name conflicts and allow sqlite migrations#270
2-Coatl merged 6 commits intodevelopfrom
feature/explain-runserver-command-behavior-15-25-19

Conversation

@2-Coatl
Copy link
Collaborator

@2-Coatl 2-Coatl commented Nov 18, 2025

Summary

  • rename related_name fields to avoid user model conflicts across configuration and permissions modules and register the granular permission model
  • add bcrypt and scikit-learn to the base requirements and enforce unique related_name expectations with new unit coverage
  • guard SQL-heavy permission migrations and merge dashboard branches so migrations run cleanly on SQLite in development

Testing

  • python manage.py migrate --noinput
  • python manage.py check
  • pytest tests/unit/test_related_names_conflicts.py

Codex Task

Copilot AI review requested due to automatic review settings November 18, 2025 15:25
Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Copy link
Collaborator Author

@2-Coatl 2-Coatl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@copilot DEBES de modificarlo, porque esto no pasa clean code

Copy link
Contributor

Copilot AI commented Nov 18, 2025

@2-Coatl I've opened a new pull request, #278, to work on those changes. Once the pull request is ready, I'll request review from you.

[WIP] Fix related name conflicts and improve code quality
@2-Coatl 2-Coatl merged commit 46f6b6c into develop Nov 18, 2025
5 of 25 checks passed
@2-Coatl 2-Coatl deleted the feature/explain-runserver-command-behavior-15-25-19 branch November 18, 2025 21:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants