Skip to content

Update the upgrade file #1324

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

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from
Open

Update the upgrade file #1324

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 9 additions & 0 deletions docs/upgrading.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -96,3 +96,12 @@ The custom code for the message templates is saved in the following files:

**Note:**
The custom changes made to the receipt templates were implemented to meet specific project requirements and priorities. When updating CiviCRM, we need to revisit these templates and apply the necessary modifications accordingly.

### 4. Modification On Main.php of CiviCRM
Changes on `wp-content/plugins/civicrm/civicrm/CRM/Contribute/Form/Contribution/Main.php`
Text Change;
Old Text:
Comment on lines 97 to +103
Copy link

@coderabbitai coderabbitai bot Jun 26, 2025

Choose a reason for hiding this comment

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

🛠️ Refactor suggestion

Section number duplicates existing heading “4.” — increment to maintain unique ordering

There is already a heading “4. Modification on event email template”. Re-using “4.” here breaks sequencing and can mislead readers skimming the doc.
Change this new heading to “5.” (or renumber all sections with an ordered list generator).

-### 4. Modification On Main.php of CiviCRM
+### 5. Modification on Main.php of CiviCRM
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
**Note:**
The custom changes made to the receipt templates were implemented to meet specific project requirements and priorities. When updating CiviCRM, we need to revisit these templates and apply the necessary modifications accordingly.
### 4. Modification On Main.php of CiviCRM
Changes on `wp-content/plugins/civicrm/civicrm/CRM/Contribute/Form/Contribution/Main.php`
Text Change;
Old Text:
**Note:**
The custom changes made to the receipt templates were implemented to meet specific project requirements and priorities. When updating CiviCRM, we need to revisit these templates and apply the necessary modifications accordingly.
### 5. Modification on Main.php of CiviCRM
Changes on `wp-content/plugins/civicrm/civicrm/CRM/Contribute/Form/Contribution/Main.php`
Text Change;
Old Text:
🤖 Prompt for AI Agents 
In docs/upgrading.md around lines 97 to 103, the heading "4. Modification On
Main.php of CiviCRM" duplicates an existing section number "4." which breaks the
sequence. Change this heading number from "4." to "5." to maintain unique and
correct ordering of sections in the document.

$errors["price_{$priceField->id}"] = ts('Thank you for your consideration, request you to contribute a minimum of INR %1',

New Text:
$errors["price_{$priceField->id}"] = ts('Thank you for your consideration, request you to donate a minimum of INR %1',