Skip to content
This repository has been archived by the owner on Jul 1, 2022. It is now read-only.

Fix needed dependency missed for PEF GXP integration #2401

Merged
merged 1 commit into from
Feb 16, 2021
Merged

Conversation

podarok
Copy link
Contributor

@podarok podarok commented Feb 15, 2021

Original Issue, this PR is going to fix: https://openy.atlassian.net/browse/MAINTAIN-50

Make sure these boxes are checked before asking for review of your pull request - thank you!

If there is a new feature or this is a bug fix - use 9.x-2.x branch. We'll tag for release if the bug is critical asap or tag for release next bug fix release until critical issue arrived.

Steps for review

  • Enable OpenY Pef GXP Sync module.
  • Verify OpenY Mappings is enabled too.

General checks

  • All coding styles are fulfilled and there are no any issues reported by CodeSniffer. See Code of Conduct.
  • Documentation has been updated according to PR changes.
  • Steps for review have been provided according to PR changes.
    Steps for review
  • Make sure you've provided all necessary hook_update_N to support upgrade path.
  • Make sure your git email is associated with account on drupal.org, otherwise you won't get commits there.
    drupal.org email
  • If you would like to get credits on drupal.org, check documentation.

Thank you for your contribution!

@podarok podarok added Level: Novice Low effort issue Type: Bug Bug. Something is not working as intended. Type: Task Maintenance, subtasks dependencies Issues with dependencies labels Feb 15, 2021
@podarok podarok self-assigned this Feb 15, 2021
@podarok podarok changed the title Update openy_pef_gxp_sync.info.yml Fix needed dependency missed for PEF GXP integration Feb 15, 2021
@podarok
Copy link
Contributor Author

podarok commented Feb 15, 2021

retest this please

@gundevel
Copy link
Collaborator

Build comment file:

Check Open Y Installation Wizard at http://install.openy.cibox.tools/build3091/install.php


Test PASSed.
Refer to this link for build results (access rights to CI server needed):
http://openy.cibox.tools:8080/job/PR_BUILDER_COMPOSER/3091/

@gundevel
Copy link
Collaborator

Open Y Upgrade Path site check installed at http://upgrade.openy.cibox.tools/build3091/

@ymcatwincities ymcatwincities deleted a comment from gundevel Feb 15, 2021
@ymcatwincities ymcatwincities deleted a comment from gundevel Feb 15, 2021
@ymcatwincities ymcatwincities deleted a comment from gundevel Feb 15, 2021
@ymcatwincities ymcatwincities deleted a comment from gundevel Feb 15, 2021
@podarok
Copy link
Contributor Author

podarok commented Feb 15, 2021

retest this please

@gundevel
Copy link
Collaborator

Build comment file:

Check Open Y Installation Wizard at http://install.openy.cibox.tools/build3092/install.php


Test PASSed.
Refer to this link for build results (access rights to CI server needed):
http://openy.cibox.tools:8080/job/PR_BUILDER_COMPOSER/3092/

@gundevel
Copy link
Collaborator

Open Y Upgrade Path site check installed at http://upgrade.openy.cibox.tools/build3092/

@podarok
Copy link
Contributor Author

podarok commented Feb 15, 2021

retest this please

@gundevel
Copy link
Collaborator

Build comment file:

Check Open Y Installation Wizard at http://install.openy.cibox.tools/build3093/install.php


Test PASSed.
Refer to this link for build results (access rights to CI server needed):
http://openy.cibox.tools:8080/job/PR_BUILDER_COMPOSER/3093/

@gundevel
Copy link
Collaborator

Open Y Upgrade Path site check installed at http://upgrade.openy.cibox.tools/build3093/

@ddrozdik ddrozdik added the PR: Code Reviewed Code reviewed. Ready for QA label Feb 15, 2021
@podarok
Copy link
Contributor Author

podarok commented Feb 16, 2021

retest this please

@gundevel
Copy link
Collaborator

Build comment file:

Check Open Y Installation Wizard at http://install.openy.cibox.tools/build3094/install.php


Test PASSed.
Refer to this link for build results (access rights to CI server needed):
http://openy.cibox.tools:8080/job/PR_BUILDER_COMPOSER/3094/

@gundevel
Copy link
Collaborator

Open Y Upgrade Path site check installed at http://upgrade.openy.cibox.tools/build3094/

@podarok podarok merged commit 5e819be into 9.x-2.x Feb 16, 2021
@podarok podarok deleted the 9.x-2.x-gxp branch February 16, 2021 11:25
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
dependencies Issues with dependencies Level: Novice Low effort issue PR: Code Reviewed Code reviewed. Ready for QA Type: Bug Bug. Something is not working as intended. Type: Task Maintenance, subtasks
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants