- 🇳🇱Netherlands MegaChriz
After the release of Feeds 8.x-3.0-rc1 → , tests now pass. Merged the code!
I'm leaving the issue open for possible future automated fixes.
It would be good to test Commerce Feeds manually with the new Commerce 3.0.0-alpha1 release → before creating a new release.
-
megachriz →
committed c67ec85d on 8.x-1.x
Issue #3429338 by megachriz, Project Update Bot: Added Drupal 11...
-
megachriz →
committed c67ec85d on 8.x-1.x
- @anwar_max opened merge request.
- Issue created by @anwar_max
- @anwar_max opened merge request.
- Issue created by @anwar_max
- @anwar_max opened merge request.
- Issue created by @anwar_max
- @anwar_max opened merge request.
- Issue created by @anwar_max
- @anwar_max opened merge request.
- Issue created by @anwar_max
-
wells →
committed 2ca4489f on 3.x authored by
ankitv18 →
Issue #3434706 by ankitv18, Project Update Bot: Automated Drupal 11...
-
wells →
committed 2ca4489f on 3.x authored by
ankitv18 →
- First commit to issue fork.
- 🇧🇪Belgium BramDriesen Belgium 🇧🇪
We can't just change this, we need to update the versions to a minimum of 10.2.0. See: https://api.drupal.org/api/drupal/core%21lib%21Drupal%21Core%21Form%21Co...
- 🇧🇪Belgium BramDriesen Belgium 🇧🇪
We can't just change this, we need to update the versions to a minimum of 10.2.0. See: https://api.drupal.org/api/drupal/core%21lib%21Drupal%21Core%21Form%21Co...
- 🇧🇪Belgium BramDriesen Belgium 🇧🇪
We can't just change this, we need to update the versions to a minimum of 10.2.0. See: https://api.drupal.org/api/drupal/core%21lib%21Drupal%21Core%21Form%21Co...
Automatically closed - issue fixed for 2 weeks with no activity.
Automatically closed - issue fixed for 2 weeks with no activity.
- @wells opened merge request.
- First commit to issue fork.
Automatically closed - issue fixed for 2 weeks with no activity.
- 🇬🇭Ghana perrypierre
I tested MR #145 with a new installation of Drupal 11, and it works as expected. No issues found during installation or testing.
- 🇳🇱Netherlands MegaChriz
All checks are green. I tested the ISBN-module manually as well on Drupal 11. All seems well.
-
megachriz →
committed c7dc56d2 on 8.x-1.x
Issue #3431115 by Project Update Bot, megachriz: Added Drupal 11...
-
megachriz →
committed c7dc56d2 on 8.x-1.x
- 🇸🇮Slovenia KlemenDEV
I think the IE support removal is very good change too
- @anwar_max opened merge request.
- Issue created by @anwar_max
- @anwar_max opened merge request.
- 🇬🇧United Kingdom Eli-T Manchester
Drupal 11 support will only be added to the 2.0.x branch.
- Issue created by @anwar_max
- 🇨🇦Canada joseph.olstad
If you know of any security vulnerabilities, pleasr let me know where to find them! If they affect current supported jQuery3/4 with bootstrap 3, then they'll be dealt with, if it's for older versions only, then no.
Keep in mind, I already gutted IE support which actually went well, was pretty straight forward. - @anwar_max opened merge request.
- Issue created by @anwar_max
- @anwar_max opened merge request.
- Issue created by @anwar_max
- @anwar_max opened merge request.
- Issue created by @anwar_max
- @wells opened merge request.
- First commit to issue fork.
- @wells opened merge request.
-
wells →
committed 07d37a25 on 4.0.x authored by
ankitv18 →
Issue #3434681 by ankitv18, Project Update Bot, wells, berramou,...
-
wells →
committed 07d37a25 on 4.0.x authored by
ankitv18 →
- 🇺🇸United States wells Seattle, WA
Adding credits from 📌 Remove watchdog_exception it's deprecated Closed: outdated .
- First commit to issue fork.
- 🇪🇸Spain rodrigoaguilera Barcelona
Have you considered splitting off the webform submodule to a different project?
- 🇮🇩Indonesia el7cosmos 🇮🇩 GMT+7
Tests on D11 are still blocked on webform dependency, we need to figure out what to do with it.
One option is to remove webform from dev dependency and skip the webform tests, unless we can separate dependency for D10 and D11.
- 🇩🇪Germany diqidoq Berlin | Hamburg | New York | London | Paris
duh! yeah the core version requirements in the examples I have missed to watch after *facepalm*
-
damienmckenna →
committed 9e8ae384 on 5.1.x authored by
Project Update Bot →
Issue #3438158 by Project Update Bot, grevil, vladimiraus, ankitv18,...
-
damienmckenna →
committed 9e8ae384 on 5.1.x authored by
Project Update Bot →
- @anwar_max opened merge request.
- 🇵🇹Portugal jcnventura
@joseph.olstad Think about never actually doing any release other than alpha1, alpha2, etc. There are known security vulnerabilities that have gone unpatched for the last 5 years.. If you're going to make new versions, you need to decide if you take the responsibility to fix those vulnerabilities or if you simply ignore them. Marking them as alpha clearly signals those releases as unsupported.
- First commit to issue fork.
- @anwar_max opened merge request.
- 🇳🇱Netherlands JoshaHubbers
Hi Heikkiy,
When I look at the table here https://www.drupal.org/docs/distributions/degov/semantic-versioning-model → it suggests:
0.0.x = patch version for bugfix
0.x.0 = minor version update with new features that do not break functionality
x.0.0 = major, new features, breaking backwards compatability.Well, this upgrade doesn't really add new functionality, so I think your decision to go to 1.0.4 is totally fine ;-).
Thanks for your collaboration!
- First commit to issue fork.
Automatically closed - issue fixed for 2 weeks with no activity.
- 🇨🇷Costa Rica yuvania
Reviewed and tested the patch. Everything works as expected.
Automatically closed - issue fixed for 2 weeks with no activity.
- @yuvania opened merge request.
- First commit to issue fork.
Automatically closed - issue fixed for 2 weeks with no activity.
- 🇪🇸Spain jonhattan Plasencia
jonhattan → changed the visibility of the branch project-update-bot-only to active.
- 🇪🇸Spain jonhattan Plasencia
jonhattan → changed the visibility of the branch project-update-bot-only to hidden.
- 🇨🇭Switzerland stefanos.petrakis Biel, Switzerland
This looks good to me as well! Thanks for your help.
You are welcome!
We could drop support for `^8 ||` at this point, I guess?
Done!
-
jsacksick →
committed 081df920 on 8.x-1.x authored by
Project Update Bot →
Issue #3428414 by jsacksick: D11 & Commerce 3 support.
-
jsacksick →
committed 081df920 on 8.x-1.x authored by
Project Update Bot →
- @dqd opened merge request.
- @ao5357 opened merge request.
- 🇺🇸United States ao5357
It looks like the rector patch only added ^11 to the info yaml file. It's unclear if other Drupal 11 compatibility fixes are required against 3.x, but it's certainly worthwhile to support Drupal 11. I've manually applied the patch to the MR for ease of composer-ing.
- First commit to issue fork.
- @asamoahboateng opened merge request.
- 🇦🇺Australia tomthorp
After much module testing, I now have a "working" TVI module on Drupal 11. The patch file is attached.
At this stage, the code has not gone through regression testing. If one of the module maintainers can do this at their earliest convenience, that would be great. (PS. I am a novice when it comes to Drupal module debugging / maintenance. So any pointers in this respect would be gladly appreciated.)
- @kewinnammert opened merge request.
Automatically closed - issue fixed for 2 weeks with no activity.
- 🇵🇱Poland kewin.nammert
kewin.nammert → made their first commit to this issue’s fork.
- 🇩🇪Germany Grevil
New changes LGTM, and make the module D9 compatible once again! Thanks, @ankitv18! :)
Automatically closed - issue fixed for 2 weeks with no activity.
- 🇸🇮Slovenia KlemenDEV
Big thanks @joseph.olstad for the efforts done here! Amazing work, looking forward to the movement of this, as it would be an amazing thing not having to go to Bootstrap 5 as some of my websites would require tons of work to achieve that
- 🇩🇪Germany diqidoq Berlin | Hamburg | New York | London | Paris
Gitlab ci file has been added manually or something so that its creation is a missing step in the change and is not registered as "new file" causing merge to fail with "file not exist". It wants to add the content to the file, but create the file. It assumes it exists already, which is not.
- 🇩🇪Germany diqidoq Berlin | Hamburg | New York | London | Paris
Not addressed yet but maybe a good hint for follow ups:
Deprecated API use for APIs removed in future Drupal major versions is not required to fix yet.
web/modules/contrib/duration_field/src/Element/DurationElement.php 41 Class Drupal\duration_field\Element\DurationElement extends deprecated class Drupal\Core\Render\Element\FormElement. Deprecated in drupal:10.3.0 and is removed from drupal:12.0.0. use Drupal\Core\Render\Element\FormElementBase instead.
web/modules/contrib/duration_field/src/Element/GranularityElement.php 20 Class Drupal\duration_field\Element\GranularityElement extends deprecated class Drupal\Core\Render\Element\FormElement. Deprecated in drupal:10.3.0 and is removed from drupal:12.0.0. use Drupal\Core\Render\Element\FormElementBase instead.
- 🇨🇦Canada joseph.olstad
I've figured out how the cdn works, but I haven't added it to the docs, here it is:
css: https://cdn.jsdelivr.net/gh/entreprise7pro/bootstrap@v3-dev-2/dist/css/bootstrap.min.css
css_theme: https://cdn.jsdelivr.net/gh/entreprise7pro/bootstrap@v3-dev-2/dist/css/bootstrap-theme.min.css
js: https://cdn.jsdelivr.net/gh/entreprise7pro/bootstrap@v3-dev-2/dist/js/bootstrap.min.jsAlso made a host for the docs.
- @dqd opened merge request.
- 🇩🇪Germany diqidoq Berlin | Hamburg | New York | London | Paris
Close this as duplicate of the youngest identical upgrade bot issue 📌 Automated Drupal 11 compatibility fixes for view_entity_ref_field_formatter Needs review Maintainers: it is recommended to have a latest branch/dev release available. This works best with the upgrade bot to prevent hiccups.
- 🇩🇪Germany diqidoq Berlin | Hamburg | New York | London | Paris
Close this as duplicate of the youngest identical upgrade bot issue 📌 Automated Drupal 11 compatibility fixes for view_entity_ref_field_formatter Needs review Maintainers: it is recommended to have a latest branch/dev release available. This works best with the upgrade bot to prevent hiccups.
- 🇩🇪Germany diqidoq Berlin | Hamburg | New York | London | Paris
Close this as duplicate of the youngest identical upgrade bot issue 📌 Automated Drupal 11 compatibility fixes for view_entity_ref_field_formatter Needs review Maintainers: it is recommended to have a latest branch/dev release available. This works best with the upgrade bot to prevent hiccups.
- 🇮🇳India ankitv18
DeprecationHelper also introduced in D10.1 so it's better if we can stick with FileSystemInterface::REPLACE until the minimum support is D10.3 or later on at the time of the D12 readiness.
cc: @damienmckenna
- 🇨🇦Canada joseph.olstad
Ok I've spent some time on the fork, upgraded bootstraps javascript to accept jQuery 4. I removed nearly all microsoft IE support from the entire branch. I am able to compile all of this, it is compiling and I have compiled it and pushed it up and made a github release of bootstrap and called it v3-dev-1
the actual release once it's tested and people are happy with it will be 3.4.2Here's the zip file you can try out, with the dist folder has the compiled js and css
https://github.com/entreprise7pro/bootstrap/archive/refs/tags/v3-dev-1.zipThere's also the same thing in tar.gz
https://github.com/entreprise7pro/bootstrap/archive/refs/tags/v3-dev-1.t...Hi @jncventura, ok, do you have a github account? Perhaps I can grant you admin access to my fork.
BTW
https://github.com/entreprise7pro/bootstrap/archive/refs/tags/v3-dev-1.zipI appreciate folks helping to review this build of bootstrap against Drupal 11.
Thanks
avoiding an unnecessary drupal module/module-fork would be good.
If this module shifts from depending on and installing the original Bootstrap JS package towards depending on and installing the fork linked above, I think this should be documented and explained with thorough detail on the module home page... why it is that we needed to fork, whose behind "enterprise7pro", etc.
Automatically closed - issue fixed for 2 weeks with no activity.
- 🇵🇹Portugal jcnventura
@joseph.olstad in #23 (which is weirdly unpublished), you suggested to fork twbs/bootstrap and have the build method be a Drupal method. You should be fine with just adding GitHub Actions to the fork you just did in GitHub to generate the minified Javascript libraries. Then, the only change would be to the libraries.yml of this theme to fetch the libraries from GitHub.
There's zero need to create a Drupal module when the code you forked does not use any part of Drupal.
Automatically closed - issue fixed for 2 weeks with no activity.
Automatically closed - issue fixed for 2 weeks with no activity.
- 🇺🇸United States japerry KVUO
Merged in the previous code and new code. Looks good!
-
japerry →
committed b1b329ce on 8.x-1.x authored by
deepakkm →
Issue #3435303 by deepakkm: Automated Drupal 11 compatibility fixes for...
-
japerry →
committed b1b329ce on 8.x-1.x authored by
deepakkm →
- 🇨🇦Canada joseph.olstad
I'm working on a 3x fork, the main branch is the v4/v5 stuff, v3-dev is the 3x dev branch, I'll set this as the default branch shortly.
Fork is here:
https://github.com/entreprise7pro/bootstrap It could use
\Drupal\Core\Utility\DeprecationHelper::backwardsCompatibleCall()
and selectively useFileSystemInterface::REPLACE
orFileExists::Replace
, right?I'm not fully following the discussion here, but if it is possible to make this module D11 compatible, that would be greatly appreciated. Migrating to Bootstrap 5 would be a momentous task.
Automatically closed - issue fixed for 2 weeks with no activity.
- 🇨🇦Canada joseph.olstad
@jncventura, perhaps I'm missing something however radix is already doing something similar to what I'm suggesting we do. With that said, there's maybe a manual step that the radix project does. The Bootstrap license is an MIT License, which afaik, is compatible with Drupal licensing. Perhaps there's a nuance I'm missing here?
- 🇵🇹Portugal jcnventura
The jQuery 4 compatible version of Bootstrap should not be a Drupal extension at all. It should probably be in GitHub and a simply be a fork of the existing sources.
- 🇨🇦Canada joseph.olstad
Ok, thanks for that @jcnventura.
We'll 100% for sure have to fork twbs bootstrap and remove that version check and any IE11/IE10 specific js code, should be a fairly easy task.
Suggest that we add a contrib module , example, call it bootstrap3, in there would be the build sources and compile instrucitons, this is where we'd have the new version of bootstrap and it would generate an optimized min js that we'll consume and add as a dependency for use by the bootstrap theme.
Some refactoring involved, I am imagining that this is 100% do-able.
The radix theme actually includes bootstrap sources in it's builds, we could do something similar to what radix does but I think to align with how we've been doing things for years and to avoid hacks (radix can get messy), I'm suggesting putting it in a support module called bootstrap3 (for example).
-
joegraduate →
committed 61dc293e on 2.x
Issue #3469831: New parameter added to \Drupal\Core\Form\ConfigFormBase...
-
joegraduate →
committed 61dc293e on 2.x
- 🇵🇹Portugal jcnventura
@joseph.olstad in #10 was the point where I got this theme to work in Drupal 11. The upstream bootstrap code has a test early on that checks for the version of jQuery being used. If it detects jQuery 4, it throws an error and refuses to run.
My efforts stopped at that point, as asking a Drupal theme maintainer to co-maintain Bootstrap 3 is simply too much to ask (even if it is maintaining a patch and not the full library, I think it will eventually become the full library).
We have 2 years to migrate away from Bootstrap 3. I think the best solution here would be a tutorial on the best way to migrate from this theme to another of the Bootstrap 5-based themes.
- 🇺🇸United States DamienMcKenna NH, USA
While reviewing this I realized that the change from FileSystemInterface::REPLACE to FileExists::Replace makes the module require 10.3: https://www.drupal.org/node/3426517 → I'm not sure I want to drop 9 and 10.2 entirely yet. Argh.
- 🇺🇸United States DamienMcKenna NH, USA
Would it be possible to support D9 and maybe leaving D12-level deprecations until later? I'd prefer to support D9 if possible. Thank you.
- 🇸🇮Slovenia KlemenDEV
Here is a full list of changes in jQuery 4: https://blog.jquery.com/2024/02/06/jquery-4-0-0-beta/
Maybe it is doable, maybe even this module could provide own fork of JS files of bootstrap
- @jnettik opened merge request.
- First commit to issue fork.
- 🇦🇺Australia VladimirAus Brisbane, Australia
Postponed. The following modules required to be updated first:
- 📌 Automated Drupal 11 compatibility fixes for campaignmonitor Needs work
- ✨ Drupal 11 compatibility fixes for webform Needs work
- 🇦🇺Australia VladimirAus Brisbane, Australia
vladimiraus → changed the visibility of the branch 3435621-automated-drupal-11 to hidden.
- 🇮🇳India zeeshan_khan
@benstallings - You have added "Related issues: #3465786: Add support for Drupal 11" this one is for tailwindcss project not for encrypt so I am opening it again.
Please correct me if I'm wrong. - First commit to issue fork.
-
smustgrave →
committed c08ac7d6 on 3.0.x
Issue #3438458: Automated Drupal 11 compatibility fixes for...
-
smustgrave →
committed c08ac7d6 on 3.0.x
- 🇺🇸United States shelane
We'll have to change support to a minimum of 10.2 for this. I don't have a problem with that. In some cases a new branch/version has been done for this and in some cases it has stayed on the same. If we ever do need to fix anything with a D9 compatible version, then we will need to cut a new branch and 2.0 version.
- Issue created by @barbarae
- 🇩🇪Germany Grevil
Made some final adjustments. @rajeshreeputra are you able to merge issues as well? Or are we still waiting for @bohart? Should we contact him via mail?
- @benstallings opened merge request.
-
liam morland →
committed 1bc2ca70 on 5.2.x
Issue #3430593: Update function signature in call to ConfigFormBase::...
-
liam morland →
committed 1bc2ca70 on 5.2.x
-
borisson_ →
committed d47b5d48 on 8.x-1.x authored by
Project Update Bot →
Issue #3434389: Automated Drupal 11 compatibility fixes for...
-
borisson_ →
committed d47b5d48 on 8.x-1.x authored by
Project Update Bot →
- First commit to issue fork.
- 🇩🇪Germany Grevil
The test failures seem unrelated, since tests have not run automatically for a while now.
Automatically closed - issue fixed for 2 weeks with no activity.
- First commit to issue fork.
Automatically closed - issue fixed for 2 weeks with no activity.
- 🇺🇸United States greggles Denver, Colorado, USA
This looks good to me as well! Thanks for your help.
We could drop support for `^8 ||` at this point, I guess?
- Issue created by @glaze
- 🇨🇭Switzerland stefanos.petrakis Biel, Switzerland
Added the GitlabCI yml file, phpunit tests are green, other non-critical tests are failing but that can be fixed in a follow-up issue.
I will move this to RTBC and would like to merge in the next 1-2 days unless a review comes in that blocks that action.
It would be good to have both D11 support as well as (GitlabCI) tests running again.
Automatically closed - issue fixed for 2 weeks with no activity.
- 🇩🇪Germany Anybody Porta Westfalica
Can be closed now that D11 is out. Thank you SO MUCH!!
- 🇨🇭Switzerland stefanos.petrakis Biel, Switzerland
Picking this up and hijacking it to add the GitlabCI integration
Automatically closed - issue fixed for 2 weeks with no activity.
Automatically closed - issue fixed for 2 weeks with no activity.
Automatically closed - issue fixed for 2 weeks with no activity.
Automatically closed - issue fixed for 2 weeks with no activity.
-
marcoscano →
committed 9f94f146 on 1.x authored by
Project Update Bot →
Issue #3431910 by marcoscano, minirobot: Automated Drupal 11...
-
marcoscano →
committed 9f94f146 on 1.x authored by
Project Update Bot →
Automatically closed - issue fixed for 2 weeks with no activity.
Automatically closed - issue fixed for 2 weeks with no activity.
Automatically closed - issue fixed for 2 weeks with no activity.
Automatically closed - issue fixed for 2 weeks with no activity.