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.

Sign up for GitHub

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

Jump to bottom

Fix upgrade version for #13487 #15031

Merged
merged 1 commit into from
Aug 13, 2019
Merged

Fix upgrade version for #13487 #15031

merged 1 commit into from
Aug 13, 2019

Conversation

mfb
Copy link
Contributor

@mfb mfb commented Aug 13, 2019

Overview

#13487 referenced the wrong version in its schema update.

@civibot civibot bot added the master label Aug 13, 2019
@civibot
Copy link

civibot bot commented Aug 13, 2019

(Standard links)

@eileenmcnaughton
Copy link
Contributor

@mfb @seamuslee001 does this have any practical impact - if so we should target the rc

@mfb
Copy link
Contributor Author

mfb commented Aug 13, 2019

I have no idea what these XML elements do but I thought it should be correct :)

@seamuslee001
Copy link
Contributor

I don't think there is any practical impact of this specific field

@eileenmcnaughton eileenmcnaughton merged commit 25958ba into civicrm:master Aug 13, 2019
@eileenmcnaughton
Copy link
Contributor

OK master is fine then

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
master
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@mfb @eileenmcnaughton @seamuslee001