Account created on 1 February 2012, over 13 years ago
#

Merge Requests

More

Recent comments

🇺🇸United States dcam

Yes, that's the follow-up. @smustgrave quoted @berdir from #17 in 📌 Don't delete view due to config dependencies being deleted Active . I'm setting this back to Fixed.

🇺🇸United States dcam

I improved the test. I think it's probably good to go now.

🇺🇸United States dcam

Since one of the other library split issues got committed this one needed a rebase with updated performance metrics. I also updated the deprecation version, but I'm not going to roll back the RTBC status for that, especially since @alexpott allowed self-RTBCing in the other issue for the same minor update.

This issue can be added to the CR I made yesterday for the 11.3.0 splits: [#3530832].

🇺🇸United States dcam

I created a new CR for libraries moved in 11.3.0 and made the requested changes. I'm restoring the RTBC status per #24.

🇺🇸United States dcam

And now you can see that multiple content translation UI tests are failing. Thank you @godotislate!

🇺🇸United States dcam

I asked in the core-development Slack if the test-only job passes because the new assertions are in an abstract base class. Per @godotislate that is the case. The test-only runner looks for changes in *Test.php files. I was advised to create a test-only MR, which I did in MR 12393.

🇺🇸United States dcam

I'll argue that a module isn't "good to go" if its automated tests aren't running, which was the case here because a .gitlab-ci.yml hadn't been added yet. I added one and am setting the status back to Needs Review. A maintainer needs to review the testing configuration and enable/disable options that they want to support.

I also removed Drupal 9 from all of the info.yml Core compatibility strings. GitLab CI can't test against it, so you can't guarantee compatibility. A breaking change may be introduced and you wouldn't know until someone filed a bug report unless you're running the tests yourself. I've done this in the modules I maintain and encourage others to do the same. I typically release a new minor version when doing this. It isn't necessary to release a new major version.

As for everyone else asking for this to be committed: I checked the most recently-active maintainer's user profile page. He mentions that he's been serving in the Ukrainian National Guard. So I wouldn't expect a fast response. My own plan is to copy the module and this compatibility fix to a private GitHub mirror until an update is made. I'm not interested in maintaining the module because I know nothing about it or the Mailgun service yet. This is my first day of using it.

🇺🇸United States dcam

dcam made their first commit to this issue’s fork.

🇺🇸United States dcam

I'm able to reproduce the issue. You don't need to use views_embed_view(). It happens with a regular, old Page display.

It can only be seen in the raw HTML, which is also what a validator looks at. When the FullCalendar JS is executed it replaces the offending DIVs.

I tried to figure out how to solve it too, but didn't get anywhere. Since the problem doesn't exist after the JS runs, then one could argue there's no problem. But I empathize with anyone who doesn't want to have to deal with validation errors on their pages, especially if they have to explain them to supervisors.

🇺🇸United States dcam

I completely missed seeing that this was implemented under the Calendar Appearance/Sizing settings.

🇺🇸United States dcam

Since the schema issue was merged I decided to make an MR. It only took a few minutes of work.

If it's decided to move this forward then I'll make a change record. And the lifecycle_link in the info file will have to be updated. Just set the issue back to NW with a note.

I wasn't certain if it's better to make an update function to uninstall fullcalendar_legend or let site builders do it themselves. I decided to err on the side of not uninstalling something that custom projects might depend on. What are your thoughts?

I left the issue version as 3.0.x to get the latest code changes, but per the semver rules this will force the release of 3.1.0.

🇺🇸United States dcam

Sorry, I'm unable to review this because I can't reproduce the issue. I set up a new content type, added a single-valued datetime field, generated content, and created a few that uses AJAX. I don't get an AJAX error and nothing appears in my logs.

Am I missing a step?

🇺🇸United States dcam

All linting tests are green now.

🇺🇸United States dcam

Glad you caught that. Thank you for doing a thorough review of it.

🇺🇸United States dcam

I'm willing to do the work if it's decided that the issue should move forward.

🇺🇸United States dcam

For the record, this updated hook is working for me with the implementation I'd already set up to give end times to events that only had start dates - after updating the implementation of course.

🇺🇸United States dcam

This ended up being more complicated than I expected. The inherent problem is that the hook which came from D7 passed \DateTime objects. But the modern code mostly deals with datetime strings. It wouldn't be quite as much of a problem if it weren't for the all-day handling. In the end, the simplest thing to do was to change the hook's definition to have the date strings passed instead of objects. We could try to convert everything to use objects instead, but it would force a major refactoring of the code, I think. It makes me wonder if this is how the hook got lost in the first place.

In case you're curious, I think the reason I left the dates in their array was due to a strict reading of the ModuleHandler::alter() docs. It only allows three parameters to be passed to implementations: $data, $context1, and $context2. I'm a very literal person, so I thought "Oh, I should put both dates in the $data variable. And that works out because the entity and fields can be the context variables." But I don't think it really matters.

🇺🇸United States dcam

dcam changed the visibility of the branch 3.0.x to hidden.

🇺🇸United States dcam

I'll release version 4.2.2 shortly.

🇺🇸United States dcam

Yep. That's a problem. Thank you for reporting it. I'll work on it.

🇺🇸United States dcam

dcam changed the visibility of the branch 3.0.x to hidden.

🇺🇸United States dcam

dcam changed the visibility of the branch 3511172-missing-schema-error to hidden.

🇺🇸United States dcam

This issue is a duplicate of 🐛 Single date field shows as long event. Active .

🇺🇸United States dcam

I've volunteered to help clean up the FullCalendar issue queue. As part of that effort I'm closing issues open for unsupported versions of the module.

If there's still interest in integrating this functionality into the primary module, then feel free to re-open it and update the version number.

🇺🇸United States dcam

Closing as a duplicate of #2833300: Customize fields title, url don't works . I suggest updating existing issues rather than opening new ones if you have changes to contribute.

🇺🇸United States dcam

I've volunteered to help clean up the FullCalendar issue queue. As part of that effort I'm closing issues open for unsupported versions of the module as "Closed (outdated)".

If this issue is still relevant for version 3.x of FullCalendar, then feel free to re-open it and update the issue version number.

🇺🇸United States dcam

I've volunteered to help clean up the FullCalendar issue queue. As part of that effort I'm closing issues open for unsupported versions of the module.

If there's still interest in adding this functionality to the module, then feel free to re-open it and update the version number.

🇺🇸United States dcam

I've volunteered to help clean up the FullCalendar issue queue. As part of that effort I'm closing issues open for unsupported versions of the module as "Closed (outdated)".

If this issue is still relevant for version 3.x of FullCalendar, then feel free to re-open it and update the issue version number.

🇺🇸United States dcam

I've volunteered to help clean up the FullCalendar issue queue. As part of that effort I'm closing issues open for unsupported versions of the module as "Closed (outdated)".

If this issue is still relevant for version 3.x of FullCalendar, then feel free to re-open it and update the issue version number.

🇺🇸United States dcam

I've volunteered to help clean up the FullCalendar issue queue. As part of that effort I'm closing issues open for unsupported versions of the module as "Closed (outdated)".

If this issue is still relevant for version 3.x of FullCalendar, then feel free to re-open it and update the issue version number.

🇺🇸United States dcam

I've volunteered to help clean up the FullCalendar issue queue. As part of that effort I'm closing issues open for unsupported versions of the module".

If this issue is still relevant for version 3.x of FullCalendar, then feel free to re-open it and update the issue version number.

🇺🇸United States dcam

I've volunteered to help clean up the FullCalendar issue queue. As part of that effort I'm closing issues open for unsupported versions of the module as "Closed (outdated)".

If this issue is still relevant for version 3.x of FullCalendar, then feel free to re-open it and update the issue version number.

🇺🇸United States dcam

I've volunteered to help clean up the FullCalendar issue queue. As part of that effort I'm closing issues open for unsupported versions of the module as "Closed (outdated)".

If this issue is still relevant for version 3.x of FullCalendar, then feel free to re-open it and update the issue version number.

🇺🇸United States dcam

I've volunteered to help clean up the FullCalendar issue queue. As part of that effort I'm closing issues open for unsupported versions of the module as "Closed (outdated)".

If this issue is still relevant for version 3.x of FullCalendar, then feel free to re-open it and update the issue version number.

🇺🇸United States dcam

I've volunteered to help clean up the FullCalendar issue queue. As part of that effort I'm closing issues open for unsupported versions of the module as "Closed (outdated)".

If this issue is still relevant for version 3.x of FullCalendar, then feel free to re-open it and update the issue version number.

🇺🇸United States dcam

I've volunteered to help clean up the FullCalendar issue queue. As part of that effort I'm closing issues open for unsupported versions of the module as "Closed (outdated)".

If this issue is still relevant for version 3.x of FullCalendar, then feel free to re-open it and update the issue version number.

🇺🇸United States dcam

I've volunteered to help clean up the FullCalendar issue queue. As part of that effort I'm closing issues open for unsupported versions of the module.

If there's still interest in integrating this functionality into the primary module, then feel free to re-open it and update the version number.

🇺🇸United States dcam

fullcalendar_field_is_date() has undergone extensive modifications during the intervening years. But I read through the function real quick and it looks like the proposed changes were implemented, albeit in a slightly different way. If I'm wrong, then feel free to re-open the issue and update the version number.

🇺🇸United States dcam

castNestedValues() no longer exists. If this issue is still relevant for version 3, then feel free to re-open it and change the version number.

🇺🇸United States dcam

The all_day code has been modified extensively since this issue was introduced. If the problem still exists in version 3, then feel free to re-open the issue and update the version number.

🇺🇸United States dcam

This needs to be triaged by a current maintainer to see if it's something they still want to do. It seems like a reasonable request, so I'm not closing it as part of my outdated issues audit. I also checked the FullCalendar JS docs and it appears that themeSystem is still supported. I'm setting the issue back to Active status because I assume that the patch will no longer apply. Also, it was mentioned in #5 that the setting's value isn't saved. It's easy to see why: the setting isn't added to the plugin configuration. So more work needs to be done.

🇺🇸United States dcam

This needs to be triaged by a current maintainer to see if it's something they still want to do. But it sounds reasonable, so I decided to not close it as part of my outdated issue audit. I'm setting the status back to active since I'm assuming that the existing patch will not apply.

🇺🇸United States dcam

The code altered by this patch has changed. If the inherent issue still exists, then feel free to re-open the issue and update the version number.

🇺🇸United States dcam

Closing this as outdated because apparently the ESLint objects to including "use strict" in contrib. See 📌 Remove "use strict" statements from JS files Active .

🇺🇸United States dcam

I have this problem in the calendar I set up too. Those events only have start dates because they represent a single moment in time, not a duration.

To solve the problem I used the fix from 🐛 Hook not firing Active to add hook_fullcalendar_process_dates_alter() in a custom module and give all dates a duration of 1hr.

🇺🇸United States dcam

Closing this as outdated since hook_fullcalendar_classes() and hook_fullcalendar_classes_alter() exist.

🇺🇸United States dcam

fullcalendar_event was removed from hook_theme() and I don't see a fullcalendar theme function in the history at all.

🇺🇸United States dcam

According to the issue priority descriptions this qualifies as Major at the most.

🇺🇸United States dcam

This is still relevant. There's no config schema in the fullcalendar_legend submodule.

🇺🇸United States dcam

I've volunteered to help clean up the FullCalendar issue queue. As part of that effort I'm closing issues open for unsupported versions of the module as "Closed (outdated)".

If this issue is still relevant for version 3.x of FullCalendar, then feel free to re-open it and update the issue version number.

🇺🇸United States dcam

I've volunteered to help clean up the FullCalendar issue queue. As part of that effort I'm closing issues open for unsupported versions of the module as "Closed (outdated)".

If this issue is still relevant for version 3.x of FullCalendar, then feel free to re-open it and update the issue version number.

🇺🇸United States dcam

I've volunteered to help clean up the FullCalendar issue queue. As part of that effort I'm closing issues open for unsupported versions of the module as "Closed (outdated)".

If this issue is still relevant for version 3.x of FullCalendar, then feel free to re-open it and update the issue version number.

🇺🇸United States dcam

I've volunteered to help clean up the FullCalendar issue queue. As part of that effort I'm closing issues open for unsupported versions of the module as "Closed (outdated)".

If this issue is still relevant for version 3.x of FullCalendar, then feel free to re-open it and update the issue version number.

🇺🇸United States dcam

Integration was added and also @mandclu took over maintenance of this module. =P

🇺🇸United States dcam

fullcalendar_legend.module no longer exists. The functionality has been moved into a Views area plugin.

🇺🇸United States dcam

From the stack trace it looks like this was triggered by hook_requirements(), which has been removed along with the rest of the .install file.

🇺🇸United States dcam

I've volunteered to help clean up the FullCalendar issue queue. As part of that effort I'm closing issues open for unsupported versions of the module as "Closed (outdated)".

#7 makes it sound like this was possibly caused by an issue in Drupal Core. If this issue is still relevant for version 3.x of FullCalendar, then feel free to re-open it and update the issue version number.

🇺🇸United States dcam

I've volunteered to help clean up the FullCalendar issue queue. As part of that effort I'm closing issues open for unsupported versions of the module as "Closed (outdated)".

If this issue is still relevant for version 3.x of FullCalendar, then feel free to re-open it and update the issue version number.

🇺🇸United States dcam

I've volunteered to help clean up the FullCalendar issue queue. As part of that effort I'm closing issues open for unsupported versions of the module as "Closed (outdated)".

If this issue is still relevant for version 3.x of FullCalendar, then feel free to re-open it and update the issue version number.

🇺🇸United States dcam

I've volunteered to help clean up the FullCalendar issue queue. As part of that effort I'm closing issues open for unsupported versions of the module as "Closed (outdated)".

If this issue is still relevant for version 3.x of FullCalendar, then feel free to re-open it and update the issue version number.

🇺🇸United States dcam

The .install file and the hook_requirements() were removed.

🇺🇸United States dcam

I closed the other two issues because I doubt their patches are relevant anymore. This issue can start the effort to fix the problem with a clean slate.

🇺🇸United States dcam

I'm closing this issue in favor of #3256797: Fullcalendar is incompatible with Date Recur 3.x only because FullCalendar has changed so much that these patches are likely to be irrelevant.

🇺🇸United States dcam

I'm closing this issue in favor of #3256797: Fullcalendar is incompatible with Date Recur 3.x only because FullCalendar has changed so much that these patches are likely to be irrelevant.

🇺🇸United States dcam

The configuration page was removed, which I suspect was the cause of this issue.

🇺🇸United States dcam

fullcalendar_legend.module no longer exists. The functionality was moved into a Views area plugin.

🇺🇸United States dcam

I rebased it to see if that might fix the PhpStan errors, but it didn't. No need to grant credit to me for that. I'd recommend committing the change and backporting to 3.0.x despite the failure. There's an open issue for fixing PhpStan failures.

🇺🇸United States dcam

🐛 Module configure link in .info.yml file. Active was marked a duplicate of this issue. Identical work was done on an MR in that issue. It's recommended to grant credit to those contributors here.

I verified the bug and tested the fix. The settings form was eliminated a while ago. This bit of leftover YAML needs to get cleaned up. RTBC+1.

🇺🇸United States dcam

Marking as a duplicate of 🐛 Warning message on installation because settings form does not exist Active . I'll recommend that credit be granted to contributors in this issue.

🇺🇸United States dcam

The jquery-ui dependencies were dropped.

🇺🇸United States dcam

v3 no longer requires this kind of manual Composer configuration.

🇺🇸United States dcam

README.txt was removed.

🇺🇸United States dcam

Development has restarted on this module and version 3 was released.

🇺🇸United States dcam

This configuration no longer exists.

🇺🇸United States dcam

I've volunteered to help clean up the FullCalendar issue queue. As part of that effort I'm closing issues open for unsupported versions of the module as "Closed (outdated)".

If this issue is still relevant for version 3.x of FullCalendar, then feel free to re-open it and update the issue version number.

🇺🇸United States dcam

I've volunteered to help clean up the FullCalendar issue queue. As part of that effort I'm closing issues open for unsupported versions of the module as "Closed (outdated)".

If this issue is still relevant for version 3.x of FullCalendar, then feel free to re-open it and update the issue version number.

🇺🇸United States dcam

I've volunteered to help clean up the FullCalendar issue queue. As part of that effort I'm closing issues open for unsupported versions of the module as "Closed (outdated)".

If this issue is still relevant for version 3.x of FullCalendar, then feel free to re-open it and update the issue version number.

🇺🇸United States dcam

I've volunteered to help clean up the FullCalendar issue queue. As part of that effort I'm closing issues open for unsupported versions of the module as "Closed (outdated)".

If this issue is still relevant for version 3.x of FullCalendar, then feel free to re-open it and update the issue version number.

🇺🇸United States dcam

The Composer-based workflow was recently improved by the fullcalendar_io dependency.

🇺🇸United States dcam

This issue was fixed by some other commit.

🇺🇸United States dcam

I've volunteered to help clean up the FullCalendar issue queue. As part of that effort I'm closing issues open for unsupported versions of the module as "Closed (outdated)".

If this issue is still relevant for version 3.x of FullCalendar, then feel free to re-open it and update the issue version number.

🇺🇸United States dcam

I've volunteered to help clean up the FullCalendar issue queue. As part of that effort I'm closing issues open for unsupported versions of the module as "Closed (outdated)".

If this issue is still relevant for version 3.x of FullCalendar, then feel free to re-open it and update the issue version number.

🇺🇸United States dcam

This issue was fixed by some other commit.

🇺🇸United States dcam

I've volunteered to help clean up the FullCalendar issue queue. As part of that effort I'm closing issues open for unsupported versions of the module as "Closed (outdated)".

If this issue is still relevant for version 3.x of FullCalendar, then feel free to re-open it and update the issue version number.

Production build 0.71.5 2024