Skip to content

Various model.py language fixes - #650

Open
comradekingu wants to merge 2 commits into
CTPUG:masterfrom
comradekingu:patch-2
Open

Various model.py language fixes#650
comradekingu wants to merge 2 commits into
CTPUG:masterfrom
comradekingu:patch-2

Conversation

@comradekingu

Copy link
Copy Markdown
Contributor

No description provided.

Comment thread wafer/schedule/models.py Outdated

blocks = models.ManyToManyField(ScheduleBlock,
help_text=_("Blocks (days) on which this venue will be used."))
help_text=_("Blocks (days) this venue will be used."))

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I disagree with this edit, the string doesn't make any sense any more.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Blocks this venue will be used on?

Comment thread wafer/schedule/models.py Outdated
Comment thread wafer/schedule/models.py Outdated

blocks = models.ManyToManyField(ScheduleBlock,
help_text=_("Blocks (days) on which this venue will be used."))
help_text=_("Blocks (days) this venue will be used."))

@comradekingu comradekingu Oct 13, 2022

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Suggested change
help_text=_("Blocks (days) this venue will be used."))
help_text=_("Occupied blocks (days) for the venue."))

I like this a bit better.

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