Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

5.2.0 release notes #12100

Merged
merged 9 commits into from
Jun 6, 2018
Merged

5.2.0 release notes #12100

merged 9 commits into from
Jun 6, 2018

Conversation

agh1
Copy link
Contributor

@agh1 agh1 commented May 8, 2018

No description provided.


- **Fix for check number no longer exposed dev/core/issues/60 ([12059](https://github.com/civicrm/civicrm-core/pull/12059))**

- **Mark Accent String Tests as incomplete if on a database that is not utf8 ([12060](https://github.com/civicrm/civicrm-core/pull/12060))**
Copy link
Contributor

Choose a reason for hiding this comment

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

@agh1 I would probably remove this one as its test specific @eileenmcnaughton

@totten
Copy link
Member

totten commented Jun 5, 2018

@agh1 are you able to resolve the conflict in release-notes.md?

adds prefixing into the function used to add the columns. Approach is taken
from extended reports.

- **[dev/translation#9](https://lab.civicrm.org/dev/translation/issues/9) Create
Copy link
Contributor

Choose a reason for hiding this comment

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

this went into 5.1.2 in the end

@agh1 agh1 force-pushed the 5.2.0-releasenotes branch from ab30b1a to 5edf2d9 Compare June 6, 2018 02:10
@agh1 agh1 changed the title [WIP] 5.2.0 release notes 5.2.0 release notes Jun 6, 2018
@agh1
Copy link
Contributor Author

agh1 commented Jun 6, 2018

@totten @eileenmcnaughton @colemanw conflict resolved, should be ready to go much thanks to work from @alifrumin.

@eileenmcnaughton eileenmcnaughton merged commit 00eaf4d into civicrm:5.2 Jun 6, 2018
@totten
Copy link
Member

totten commented Jun 6, 2018

Cheers @alifrumin @agh1 !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants