Skip to content
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

Modernize package #280

Merged
merged 2 commits into from
Mar 18, 2024
Merged

Conversation

samamorgan
Copy link
Contributor

Reverts #279

Copy link
Contributor

@mschoettle mschoettle left a comment

Choose a reason for hiding this comment

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

LGTM, added a couple minor comments

Comment on lines 53 to 60
migrations.AlterIndexTogether(
name='crudevent',
index_together=set(),
),
migrations.AddIndex(
model_name='crudevent',
index=models.Index(fields=['object_id', 'content_type'], name='easyaudit_c_object__82020b_idx'),
),
Copy link
Contributor

Choose a reason for hiding this comment

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

This might not be necessary once #258 is merged

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah it won't, but I'll solve the merge conflict once it lands.

pyproject.toml Outdated Show resolved Hide resolved
@jheld
Copy link
Collaborator

jheld commented Mar 18, 2024

@samamorgan looking good. But I'm not sure what the issue with models.py is, can you check into it?

@samamorgan
Copy link
Contributor Author

@samamorgan looking good. But I'm not sure what the issue with models.py is, can you check into it?

Fixed the issues, including adding a new migration.

@jheld jheld merged commit 77b4d51 into soynatan:master Mar 18, 2024
6 checks passed
@samamorgan samamorgan deleted the revert-279-revert-267-master branch March 18, 2024 21:03
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.

3 participants