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

splitting vwis into vis and written #137

Merged
merged 1 commit into from
Jan 14, 2025
Merged

splitting vwis into vis and written #137

merged 1 commit into from
Jan 14, 2025

Conversation

acholyn
Copy link
Collaborator

@acholyn acholyn commented Jan 8, 2025

Visual & Written Influences should be 2 separate models

@acholyn acholyn requested a review from tcouch January 8, 2025 13:22
extra = 1
classes = [
"grp-collapse",
"grp-open",
]
verbose_name = "Visual & Written Influences"
verbose_name_plural = "Visual & Written Influences"
verbose_name = "Visual Influences"
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
verbose_name = "Visual Influences"
verbose_name = "Visual Influence"

On the model you've defined verbose name as "Visual Influence" rather than "Visual Influences".

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

yep, deliberate, since they're separate now I thought they might end up making new entries for each

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

worst case I'm wrong and we change it back

"grp-collapse",
"grp-open",
]
verbose_name = "Written Influences"
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
verbose_name = "Written Influences"
verbose_name = "Written Influence"

As above. Unless this is deliberate?

@acholyn acholyn merged commit 4742d33 into development Jan 14, 2025
3 checks passed
@acholyn acholyn deleted the split-vwi branch January 14, 2025 09:08
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