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

MPDX-7864 - Task phases functionailty #949

Merged
merged 87 commits into from
Oct 2, 2024
Merged

MPDX-7864 - Task phases functionailty #949

merged 87 commits into from
Oct 2, 2024

Conversation

dr-bizz
Copy link
Contributor

@dr-bizz dr-bizz commented Jun 4, 2024

Description

This pull request integrates the Task phases functionality into the MPDX application. The update encompasses several areas, including task initialization, contact management, and coaching updates.

Authors: @caleballdrin & @dr-bizz

Detailed Changes

Initialization

  1. Task Phases Functions: Set up foundational functions and hooks for task phases.
  2. Common Functions & Components: Updated common functions, components, and tests. Pulled phase data into GraphQL constants and improved error handling.
  3. Dashboard, Search, Reports & Tools: Updated to include new phase data and fixed related tests.

Contacts

  1. Dynamic Statuses: Replaced hard-coded statuses with dynamic data.
  2. Contact Details & Modals: Updated components to use new hooks, created new modals, and enhanced contact flows.
  3. Task Logging: Added functionality to log or create tasks upon contact view changes.

Tasks

  1. Task Modals & Helper Functions: Created and edited inputs for task modals, added helper functions, and integrated phase data into the task page.

Coaching

  1. API Updates: Implemented API changes, fixed spelling errors, and updated mocks.
  2. Coaching Details: Enhanced coaching details with task phase and contact statuses.

Testing

  • Comprehensive updates to related tests, ensuring all changes are seamlessly integrated and properly validated.

Note on Branch Change

Due to the high number of commits (140) on the previous branch "MPDX-task-phases" and the resulting merge conflicts from multiple code changes, I had to move to a new branch "MPDX-task-phases-new." This new branch was created to make it easier to create and review new commits without the complexities of resolving numerous conflicts.

This approach should provide a clearer and more manageable review process.

Checklist:

  • I have given my PR a title with the format "MPDX-(JIRA#) (summary sentence max 80 chars)"
  • I have applied the appropriate labels. (Add the label "On Staging" to get the branch automatically merged into staging.)
  • I have requested a review from another person on the project

Copy link
Contributor

github-actions bot commented Jun 4, 2024

Bundle sizes [mpdx-react]

Compared against 283ba6a

Route Size (gzipped) Diff
/_app 373.67 KB +1.96 KB
/accountLists 119.32 KB -1.19 KB
/accountLists/[accountListId] 270.98 KB -1016 B
/accountLists/[accountListId]/coaching/[coachingId] 247.96 KB +1.61 KB
/accountLists/[accountListId]/contacts/flows/setup 180.69 KB +32.86 KB
/accountLists/[accountListId]/reports/coaching 247.91 KB +1.6 KB
/accountLists/[accountListId]/reports/designationAccounts 243.97 KB +2.47 KB
/accountLists/[accountListId]/reports/expectedMonthlyTotal 111.05 KB +2.44 KB
/accountLists/[accountListId]/reports/partnerCurrency/[[...contactId]] 135.86 KB +1.94 KB
/accountLists/[accountListId]/reports/partnerGivingAnalysis/[[...contactId]] 135.23 KB +2.39 KB
/accountLists/[accountListId]/reports/responsibilityCenters 245.05 KB +2.47 KB
/accountLists/[accountListId]/reports/salaryCurrency/[[...contactId]] 135.86 KB +1.94 KB
/accountLists/[accountListId]/settings/admin 137.16 KB +1.29 KB
/accountLists/[accountListId]/settings/manageCoaches 141.31 KB +1.23 KB
/accountLists/[accountListId]/settings/notifications 138.97 KB +1.43 KB
/accountLists/[accountListId]/settings/organizations 138.94 KB +1.26 KB
/accountLists/[accountListId]/settings/organizations/accountLists 140.89 KB +1.47 KB
/accountLists/[accountListId]/settings/organizations/contacts 137.68 KB +1.47 KB
/accountLists/[accountListId]/tasks/[[...contactId]] 139.71 KB +1.87 KB
/accountLists/[accountListId]/tools 142.28 KB -1.12 KB
/accountLists/[accountListId]/tools/appeals/appeal/[[...appealId]] 218.07 KB +1.65 KB
/accountLists/[accountListId]/tools/fix/sendNewsletter/[[...contactId]] 117.35 KB -1.2 KB
/accountLists/[accountListId]/tools/import/csv 106.89 KB -1.2 KB
/accountLists/[accountListId]/tools/import/google 156.3 KB -2.3 KB
/accountLists/[accountListId]/tools/import/tnt 149.95 KB -2.83 KB
/login 119.5 KB -1.01 KB
Dynamic import Size (gzipped) Diff
../src/components/Contacts/ContactFlow/DynamicContactFlow.tsx -> ./ContactFlow 67.54 KB +6.44 KB
../src/components/Contacts/ContactsList/DynamicContactsList.tsx -> ./ContactsList 44.75 KB +1.85 KB
../src/components/Contacts/ContactsMap/DynamicContactsMap.tsx -> ./ContactsMap 32.43 KB +1.61 KB
../src/components/Contacts/ContactsMap/DynamicContactsMapPanel.tsx -> ./ContactsMapPanel 8.05 KB +1.8 KB
../src/components/Contacts/MassActions/Exports/Emails/DynamicMassActionsExportEmailsModal.tsx -> ./MassActionsExportEmailsModal 2.08 KB -1.02 KB
../src/components/Dashboard/ThisWeek/NewsletterMenu/MenuItems/LogNewsLetter/DynamicLogNewsletter.tsx -> ./LogNewsletter 89.77 KB -1.17 KB
../src/components/Dashboard/ThisWeek/WeeklyActivity/WeeklyReportModal/DynamicWeeklyReportModal.tsx -> ./WeeklyReportModal 33.15 KB -1.39 KB
../src/components/Layouts/Primary/TopBar/Items/AddMenu/Items/CreateMultipleContacts/DynamicCreateMultipleContacts.tsx -> ./CreateMultipleContacts 76.62 KB +1.49 KB
../src/components/Shared/Filters/DynamicFilterPanel.tsx -> ./FilterPanel 108.08 KB +2.55 KB
../src/components/Task/Modal/Form/Complete/DynamicTaskModalCompleteForm.tsx -> ./TaskModalCompleteForm 111.19 KB +12.53 KB
../src/components/Task/Modal/Form/DynamicTaskModalForm.tsx -> ./TaskModalForm 119.29 KB +12.73 KB
../src/components/Task/Modal/Form/LogForm/DynamicTaskModalLogForm.tsx -> ./TaskModalLogForm 149.91 KB +9.78 KB
../src/components/Tool/Appeal/Flow/DynamicContactFlow.tsx -> ./ContactFlow 67.54 KB +6.44 KB
../src/components/Tool/Appeal/List/ContactsList/DynamicContactsList.tsx -> ./ContactsList 44.75 KB +1.85 KB
../src/components/Tool/Appeal/Modals/UpdateDonationsModal/DynamicUpdateDonationsModal.tsx -> ./UpdateDonationsModal 103.91 KB -1.58 KB

dr-bizz added 17 commits June 4, 2024 16:22
- Pull phase data into Constants graphql
- Update test wrapper to handle and report errors
- Allow for more customization on common modal
…s well as fixing their tests

- Updating search with new Phase data
- Fixing tests in reports
- Updating settings test with new Phase data
- Update tools to use new phase hooks
- Show default views if no flows saved
- Reset to default views modal
@dr-bizz dr-bizz force-pushed the MPDX-task-phases-new branch from a8858b7 to 8016eeb Compare June 4, 2024 20:26
@dr-bizz dr-bizz changed the title Task phases functionailty MPDX-7864 - Task phases functionailty Jun 4, 2024
@dr-bizz dr-bizz added the On Staging Will be merged to the staging branch by Github Actions label Jun 4, 2024
@caleballdrin caleballdrin self-assigned this Jun 7, 2024
@dr-bizz dr-bizz added the Preview Environment Add this label to create an Amplify Preview label Jun 11, 2024
Copy link
Contributor

Preview branch generated at https://MPDX-task-phases-new.d3dytjb8adxkk5.amplifyapp.com

@dr-bizz
Copy link
Contributor Author

dr-bizz commented Sep 27, 2024

I broke the Task action in this commit 729441d - fixing now

@dr-bizz dr-bizz merged commit 837a016 into main Oct 2, 2024
17 of 18 checks passed
@dr-bizz dr-bizz deleted the MPDX-task-phases-new branch October 2, 2024 15:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
On Staging Will be merged to the staging branch by Github Actions Preview Environment Add this label to create an Amplify Preview
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants