Nicaragua
Account created on 7 August 2011, almost 14 years ago
#

Merge Requests

More

Recent comments

heddn Nicaragua

Posted on the PR but I'm not sure about the extra hardening. Other feedback addressed.

heddn Nicaragua

Can we add to an exist Functional test or create a new one that causes this?

heddn Nicaragua

If we are going to work in this space, can we also adjust SnippetTest so it no longer try/catches MigrateSkipProcessException?

heddn Nicaragua

There are still a few comments on the MR without any feedback. Also, for those facing this and don't want to apply a patch, you can always move dependencies from required to optional. I almost never use required because of reasons like what this issue causes.

heddn Nicaragua

Now this needs tests showing the problem and how it is fixed.

heddn Nicaragua

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

heddn Nicaragua

Can we add a change record with docs how to use this?

heddn Nicaragua

Given this is label only changes and tests still pass and Andy confirmed we got all the places, I'm going to go out on a limb here and mark this RTBC. Feel free to kick it back.

heddn Nicaragua

Merging what we've got. Even though this isn't done, it make incremental improvements.

heddn Nicaragua

Be nice if we could add tests and the toggle for this functionality as mentioned in the IS.

heddn Nicaragua

I've been using the module w/ Drush 13 without issue, so I'm not sure what parts of drush 13 are broken.

heddn Nicaragua

Can we add a test for this? I don't see any failing tests showing this issue. BTW, I'm certain it is a problem, we just should add tests.

heddn Nicaragua

Had a chance to manually test this today and everything seems to be in order still.

heddn Nicaragua

The lone failure was a build test failure and seems unrelated.

heddn Nicaragua

re #10:

that isn't quite true. regularly I get the access granted for the user account given for S3. and even if eventually i use an account with locked down permissions, I find it easier to setup with this drupal gui than to navigate in all of the various S3 vendor's UI and configure it.

heddn Nicaragua

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

heddn Nicaragua

Related javascript tests seem to be failing but I can't reproduce the issues manually. Any suggestions?

heddn Nicaragua

Anyone able to confirm if this makes things happy? If you do, feel free to drop in your steps to testing (similar to #31) and mark this RTBC.

heddn Nicaragua

Are we sure this is the only place that surfaces? And we should update cspell if it is.

heddn Nicaragua

Would still be nice to add a unit test.

heddn Nicaragua

Thanks for the improvements here.

heddn Nicaragua

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

heddn Nicaragua

Probably removing the string from the const is the safest fix. Feel free to open a PR.

heddn Nicaragua

Can we add a test or something to confirm the problem and fix please?

heddn Nicaragua

The fixes for this were already in the dev branch. I thought I'd released it in a tag, but must have missed it.

https://www.drupal.org/project/migrate_source_ui/releases/8.x-1.3

heddn Nicaragua

Let's merge this and tag a release. If there are any more issues, we can resolve them in a follow-up and yet another release. I'd rather not leave this hanging forever.

heddn Nicaragua

For now, I'm going to implement https://git.drupalcode.org/project/paragraphs/-/merge_requests/115/diffs... in the contrib module that this effects. I'm not sure another rewrite off all this when field migrations are about to go away is really desired.

heddn Nicaragua

Sorry, I missed that. Fixed. Thanks for your patience.

heddn Nicaragua

That label is added on d.o pages to educate visitors about what is a stable project. It can't be removed.

heddn Nicaragua

Some would kindly test and approve the latest patch.

heddn Nicaragua

This is postponed on the upstream core issue being resolved. Secondly, getting this to green on tests is proving to be troubling. I'd appreciate anyone who can help out in the space.

heddn Nicaragua

It wouldn't be trivial to back out the full phpcs changes as they were done back in January and the impact wasn't caught until recently. The easier way forward is to see these fixes land quickly.

heddn Nicaragua

🐛 Drupal 7 to Drupal 11 migration runs forever Active complains about this on the contrib project. Given it is breaking contrib, I'm moving this to a major.

heddn Nicaragua

The addition of this seems to have added an infinite loop via field discovery calling getMigrationDependencies. Will start looking for a bug report, but wanted to mention this here in case someone else has already surfaced this.

heddn Nicaragua

Ideally we identify what made this hard dependency on migrate_plus in 6.1 and revert/rollback/fix that. Is that even possible?

heddn Nicaragua

Can you tell if this makes things easier to upgrade to the 6.1.x branch?

Production build 0.71.5 2024