Skip to content
This repository has been archived by the owner on Dec 10, 2021. It is now read-only.

1.19.0 #650

Merged
merged 103 commits into from
Nov 23, 2016
Merged

1.19.0 #650

merged 103 commits into from
Nov 23, 2016

Conversation

Saphyel
Copy link
Contributor

@Saphyel Saphyel commented Nov 22, 2016

https://github.com/comicrelief/campaign/milestone/17
https://jira.comicrelief.com/secure/ReleaseNote.jspa?projectId=11402&version=10974

#646 - PLAT-608 svg search
#625 - PLAT-591 URL change news landing page *
#644 - PLAT-491 cards row title spacing
#641 - PLAT-619 Update ESU values *
#648 - RND-517 fix header kids
#636 - PLAT-614 partner row logo updates
#642 - PLAT-620 fixed merge conflict
#643 - RND-519 cards padding perc issue
#630 - RND-476 add ie9 support
#628 - PLAT-520 update esu icon
#635 - PLAT-600 Story row component updates
#629 - PLAT-594 padding options single message row
#647 - PLAT-480 ESU school phase dropdown error border
#640 - PLAT-618 ESU school phase dropdown error border
#645 - update readme IE9 support info

Heleen3258 and others added 30 commits November 10, 2016 11:56
… page and added this URL to the page restrictions of the whats going on list view block
…o PLAT-594-padding-options-single-message
…o plat-591-url-change-news-landing-page
…o plat-591-url-change-news-landing-page
…o PLAT-594-padding-options-single-message
Heleen3258 and others added 16 commits November 21, 2016 13:56
…o PLAT-594-padding-options-single-message
…o plat-480-esu-school-phase-dropdown-error-border
…gle-message

Plat 594 padding options single message row
…o plat-480-esu-school-phase-dropdown-error-border
…opdown-error-border

plat-480 ESU school phase dropdown error border
@@ -123,6 +123,8 @@
<target name="install" description="Install CR with standard configuration.">
<exec dir="${app.dir}" logoutput="true" passthru="true" checkreturn="true"
command="${drush.bin} si ${app.profile.name} -y --site-name=${app.name} --account-pass='${app.password}' --db-url=${db.querystring} ${app.profile.options}"/>
<exec dir="${app.dir}"
Copy link
Contributor

Choose a reason for hiding this comment

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

@Saphyel how come we are exporting config dir after install ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

it was discussed in the drupal 8 101 //cc @pvhee

Copy link
Contributor

Choose a reason for hiding this comment

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

my understanding was that this was a useful tool for when checking config you've changed during development, not something we need to do on every install, right ?

Copy link
Contributor Author

@Saphyel Saphyel Nov 23, 2016

Choose a reason for hiding this comment

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

@bimsonz if you change something is useful otherwise is not.. but we are dealing mainly with config changes... and we still have a lot of problems with it.

@pvhee pvhee merged commit 2aed955 into master Nov 23, 2016
@Saphyel Saphyel deleted the 1.19.0 branch November 23, 2016 16:34
@pvhee pvhee mentioned this pull request Jan 5, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants