Skip to content

Commit

Permalink
Bump info file for grant report managed entity fix
Browse files Browse the repository at this point in the history
  • Loading branch information
@seamuslee001
seamuslee001 committed Mar 10, 2022
1 parent 09ee3e5 commit d536746
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions info.xml
View file
Original file line number Diff line number Diff line change
Expand Up @@ -13,11 +13,11 @@
<author>Eileen McNaughton</author>
<email>[email protected]</email>
</maintainer>
<releaseDate>2022-03-08</releaseDate>
<version>5.12.1</version>
<releaseDate>2022-03-11</releaseDate>
<version>5.13</version>
<develStage>stable</develStage>
<compatibility>
<ver>5.46</ver>
<ver>5.47</ver>
</compatibility>
<comments>Extended reports create other report options such as crosstab urls, multiple contact custom fields, price
set reports
Expand Down

4 comments on commit d536746

@stesi561
Copy link

Choose a reason for hiding this comment

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

Is it worth dropping the compatibility version if possible? That way users will get a notification to upgrade prior to running the CiviCRM upgrade? Or will this break with earlier versions?

@seamuslee001
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@stesi561 given that Pradeeps' PR pulled out any checking on the component aspect I figured that is a compatibility break with 5.46 given the extension was only added in 5.47

@stesi561
Copy link

Choose a reason for hiding this comment

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

Aha - but we should be fine to install the extension before hand on sites without CiviGrant enabled as worst case we delete a report that doesn't do anything before we install it again.

@stesi561
Copy link

Choose a reason for hiding this comment

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

Anyway I've put up this -> https://civicrm.stackexchange.com/questions/41445/upgrade-to-5-47-x-fails-with-error-exception-api-error-db-error-constraint/ in the hope that people hitting it will find an answer quickly ! :)

Please sign in to comment.