Steisslingen
Account created on 26 July 2019, over 5 years ago
  • Founder, Senior Software Architect/Engineer at LakeDrops 
#

Merge Requests

More

Recent comments

🇩🇪Germany danielspeicher Steisslingen

Looks great. Could be merged, if pipeline succeeds.

🇩🇪Germany danielspeicher Steisslingen

Could be realted to

https://www.drupal.org/project/invoice_ninja/issues/3421977 📌 Check VAT name before syncing Active

🇩🇪Germany danielspeicher Steisslingen

Fine for me.

I think, we can close this as well: https://www.drupal.org/project/helpdesk_integration/issues/3461710 🐛 Exception when issue is not synched Active

🇩🇪Germany danielspeicher Steisslingen

I have created a MR with the changes. The build fails, but I do not know why. The branch seems up to date.

🇩🇪Germany danielspeicher Steisslingen

@jonasanne Thank you for providing this patch. We use 10.3.2 and make heavy use of E2E testing with Cypress. So we create a user who creates custom entities and after the test we delete everything. This helps a lot and it works great.

🇩🇪Germany danielspeicher Steisslingen

As discussed, this is not the right place here, see #3.

🇩🇪Germany danielspeicher Steisslingen

Will do this when I am back next weekend

🇩🇪Germany danielspeicher Steisslingen

I have added the test for the event to the existing test class, which tests all the events being fired.
For the new action, I have created a new class.

Feel free to enhance by yourself, if I missed something.
All tests are running.

🇩🇪Germany danielspeicher Steisslingen

No problem. I focus on the other issues.

🇩🇪Germany danielspeicher Steisslingen

We might, can create an event and use an ECA and/or DANSE model to dispatch.

🇩🇪Germany danielspeicher Steisslingen

@ sazevedo@ua.pt

I have version 10.3.1 and ECA 2.0 and the error is gone. For 10.3.1 (and for 1.1.7) you do not need to apply the patc anymore.

🇩🇪Germany danielspeicher Steisslingen

We need to define what to do, if issues do not get synched. This can happen with the queue in place. In Drupal the issue is stored but not in the helpdesk backend. This happens very rarely, but it can.

🇩🇪Germany danielspeicher Steisslingen

Let us postpone until a real requirement occurs.

🇩🇪Germany danielspeicher Steisslingen

Looks good. I also checked the post update script. This works as well.

🇩🇪Germany danielspeicher Steisslingen

We still get:

Invalid CI config YAML file
https://git.

drupalcode.org//-/raw//includes/includes/include.drupalci.hidden-variables.yml: Invalid configuration format

🇩🇪Germany danielspeicher Steisslingen

Now we get:

Invalid CI config YAML file
https://git.drupalcode.org//-/raw//includes/includes/include.drupalci.hidden-variables.yml: Invalid configuration format
🇩🇪Germany danielspeicher Steisslingen

We changed it accordingly. Now we get:

ERROR: Job failed: failed to pull image "drupalci/php--apache:production" with specified policies [always]: Error response from daemon: pull access denied for drupalci/php--apache, repository does not exist or may require 'docker login': denied: requested access to the resource is denied (manager.go:250:1s)

PHP version is missing.

🇩🇪Germany danielspeicher Steisslingen

It is a hard fail, because the yml file is invalid, so the pipeline does not even start.

🇩🇪Germany danielspeicher Steisslingen
include:
  - remote: https://git.drupalcode.org/${_GITLAB_TEMPLATES_REPO}/-/raw/${_GITLAB_TEMPLATES_REF}/includes/include.drupalci.main.yml
    rules:
      - if: $CI_PIPELINE_SOURCE == "merge_request_event"
        when: never
      - if: $CI_COMMIT_TAG
        when: never
      - if: $CI_COMMIT_BRANCH == "develop" || $CI_COMMIT_BRANCH =~ "/^feature\/.*/"
  - remote: https://git.drupalcode.org/${_GITLAB_TEMPLATES_REPO}/-/raw/${_GITLAB_TEMPLATES_REF}/includes/include.drupalci.variables.yml
    rules:
      - if: $CI_PIPELINE_SOURCE == "merge_request_event"
        when: never
      - if: $CI_COMMIT_TAG
        when: never
      - if: $CI_COMMIT_BRANCH == "develop" || $CI_COMMIT_BRANCH =~ "/^feature\/.*/"
  - remote: https://git.drupalcode.org/${_GITLAB_TEMPLATES_REPO}/-/raw/${_GITLAB_TEMPLATES_REF}/includes/include.drupalci.workflows.yml
    rules:
      - if: $CI_PIPELINE_SOURCE == "merge_request_event"
        when: never
      - if: $CI_COMMIT_TAG
        when: never
      - if: $CI_COMMIT_BRANCH == "develop" || $CI_COMMIT_BRANCH =~ "/^feature\/.*/"
Production build 0.71.5 2024