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

[OYPD-377] Downgrade Plugin module from 2.5 to 2.4. Add dependencies for related paragraphs. #317

Merged
merged 4 commits into from May 4, 2017

Conversation

Kerby70
Copy link
Contributor

@Kerby70 Kerby70 commented Apr 27, 2017

Jira https://propeople-us.atlassian.net/browse/OYPD-377
D.o https://www.drupal.org/node/2873442

Steps to review

  • Should see camp menu on /camps/camp-orkila, and /camps/camp-colman/about
  • Should see membership selector on /demo/join

@Kerby70
Copy link
Contributor Author

Kerby70 commented Apr 27, 2017

The failed behat test is being resolved with #311. There was an Ansible error that resolved itself when retested.

@Sanchiz
Copy link
Contributor

Sanchiz commented Apr 28, 2017

image

@Sanchiz Sanchiz added the PR: Needs Work Unfinished task. Issues still there label Apr 28, 2017
@Kerby70
Copy link
Contributor Author

Kerby70 commented Apr 28, 2017

I've rebased on root/8.x-1.x I want to at least see that WYSIWYG error happen again because manual testing shows that iframe is still on the page when that icon is clicked.

@Kerby70
Copy link
Contributor Author

Kerby70 commented Apr 28, 2017

Tests pass after rebase.
image
Perhaps we need to add an additional wait for the WYSIWYG media button before trying to switch to the iframe?

@Kerby70 Kerby70 added Status: Needs review Needs reviewer attention. and removed PR: Needs Work Unfinished task. Issues still there labels May 1, 2017
@Sanchiz Sanchiz added PR: Needs Review Needs someone review ( code ) and removed Status: Needs review Needs reviewer attention. labels May 3, 2017
@@ -12,17 +12,17 @@ entity_type: paragraph
bundle: openy_prgf_mbrshp_calc
label: Block
description: ''
required: false
required: true
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Update path for this?

@Sanchiz Sanchiz added PR: Needs Work Unfinished task. Issues still there and removed PR: Needs Review Needs someone review ( code ) labels May 3, 2017
…on\ArrayPluginDefinitionDecorator contains 1 abstract method and must therefore be declared abstract or implement the remaining methods
Copy link
Contributor

@Sanchiz Sanchiz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved.

@gundevel
Copy link
Collaborator

gundevel commented May 4, 2017

Build comment file:

===============

Environment Link
Fresh OpenY installation http://openy-dev.ffwua.com/build1079
Upgraded(upgrade path) installation http://upgrade.openy-dev.ffwua.com/build1079
Installation process http://install.openy-dev.ffwua.com/build1079/install.php

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
http://openy-dev.ffwua.com:8080/job/PR_BUILDER/1079/

@gundevel
Copy link
Collaborator

gundevel commented May 4, 2017

@Sanchiz Sanchiz merged commit 7dc4624 into ymcatwincities:8.x-1.x May 4, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
PR: Needs Work Unfinished task. Issues still there
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants