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

PLAT-97 check errors #171

Merged
merged 32 commits into from
Apr 27, 2016
Merged

PLAT-97 check errors #171

merged 32 commits into from
Apr 27, 2016

Conversation

Saphyel
Copy link
Contributor

@Saphyel Saphyel commented Apr 18, 2016

Fixes https://jira.comicrelief.com/browse/PLAT-97

Changes proposed in this pull request

  • check if dblog is "empty" after install drupal.

@Saphyel Saphyel force-pushed the feature/PLAT-97_clean_dblog branch from 60270ee to 5ddf0bb Compare April 19, 2016 16:06
@Saphyel
Copy link
Contributor Author

Saphyel commented Apr 19, 2016

@pvhee and @bendodd thanks! it worked, now we need to fix that warnings...

@pvhee
Copy link
Contributor

pvhee commented Apr 20, 2016

@gusliedke can you check the bottom of https://travis-ci.com/comicrelief/campaign/builds/24150906 pls? seems some php warnigns are generated, like Warning: file_get_contents(profiles/cr/themes/cus tom/campaign_base/js/campaign_base.min.j s): failed to open stream: No such file or directory in Drupal\Core\Asset\JsOptimizer->optimize( ) (


after_script:
- drupal-ti after_script
# Check DB errors
Copy link
Contributor

Choose a reason for hiding this comment

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

remove this comment pls, and put it in the right place

@Saphyel Saphyel mentioned this pull request Apr 21, 2016
1 task

drush wd-show --severity=critical > tmp.txt
drush wd-show --severity=warning >> tmp.txt
drush wd-show --severity=error >> tmp.txt
Copy link
Contributor

Choose a reason for hiding this comment

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

better call the file dberrors.txt or something

@wagg-matt wagg-matt merged commit 95d6ca5 into develop Apr 27, 2016
@wagg-matt wagg-matt deleted the feature/PLAT-97_clean_dblog branch April 27, 2016 15:33
@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

3 participants