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

Fix #54 - remove event name limit #167

Merged

Conversation

ak4t0sh
Copy link
Contributor

@ak4t0sh ak4t0sh commented May 27, 2024

As stated in #54

AFAIK technically mdl_event.name is a longtext and module name which is used to build the event name is a varchar(255).
I know that you can use "shortname" if you want to display a specific (and length unlimited) event name but it forces teachers to do double entry if they only want the module name to be displayed completely.

IMO truncate should be done on display only (and is correctly handle by the calendar) and not during event creation.

@danmarsden danmarsden merged commit 1df9cf1 into catalyst:MOODLE_403_STABLE May 28, 2024
7 checks passed
@danmarsden
Copy link
Member

thanks for the PR!

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