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

Feat/richer activation #79

Open
wants to merge 15 commits into
base: main
Choose a base branch
from
Open

Feat/richer activation #79

wants to merge 15 commits into from

Conversation

ter1203
Copy link
Collaborator

@ter1203 ter1203 commented Oct 9, 2024

No description provided.

Copy link

sentry-io bot commented Oct 9, 2024

🔍 Existing Issues For Review

Your pull request is modifying functions with the following pre-existing issues:

📄 File: client/src/pages/Prospecting/Prospecting.js

Function Unhandled Issue
Prospecting ReferenceError: Cannot access 'fetchPaginatedData' before initialization ...
Event Count: 1 Affected Users: 1

Did you find this useful? React with a 👍 or 👎

Copy link

netlify bot commented Oct 9, 2024

Deploy Preview for gleeful-cascaron-9766fa ready!

Name Link
🔨 Latest commit c320c20
🔍 Latest deploy log https://app.netlify.com/sites/gleeful-cascaron-9766fa/deploys/671683ff327f920009704754
😎 Deploy Preview https://deploy-preview-79--gleeful-cascaron-9766fa.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@@ -31,18 +33,20 @@ import {
getPaginatedProspectingActivities,
} from "src/components/Api/Api";
import CustomTable from "../../components/CustomTable/CustomTable";
import ProspectingMetadataOverview from "../../components/ProspectingMetadataOverview/ProspectingMetadataOverview";
import ProspectingEffortTimeline from "../../components/ProspectingEffortTimeline/ProspectingEffortTimeline";
// import ProspectingMetadataOverview from "../../components/ProspectingMetadataOverview/ProspectingMetadataOverview";
Copy link
Contributor

Choose a reason for hiding this comment

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

please remove comments

onSearch={handleSearch}
/>

<Box sx={{ flexGrow: 1, marginTop: 5 }}>
Copy link
Contributor

Choose a reason for hiding this comment

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

This seems like it can go in its own component

Copy link
Contributor

Choose a reason for hiding this comment

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

Possible or advisable to put all of the new view into an AccountDetail component?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Ok, got it

Copy link
Contributor

@NickTracker NickTracker left a comment

Choose a reason for hiding this comment

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

Looks good, just some thoughts on componentization.

onPageChange: handlePageChange,
onRowsPerPageChange: handleRowsPerPageChange,
}}
sortConfig={{
Copy link
Contributor

Choose a reason for hiding this comment

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

@ter1203 can you include this in the CustomTable within the AccountDetail compoent? You should be able to prop drill it exactly like you're doing with pagination config, when done correctly it should result in detail table rows getting sorted when you click a column.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Ok, got it.

@Nickz22
Copy link
Owner

Nickz22 commented Oct 12, 2024

@ter1203 Looks like the Account urls are redirecting to the app itself instead of Salesforce in the new table

React.App.-.12.October.2024.mp4

@Nickz22
Copy link
Owner

Nickz22 commented Oct 16, 2024

@ter1203 Looks like the Account urls are redirecting to the app itself instead of Salesforce in the new table

React.App.-.12.October.2024.mp4

@ter1203 this seems to still be occurring

Nickz22 and others added 2 commits October 18, 2024 08:31
@Nickz22 Nickz22 temporarily deployed to feat/richer-activation - insideoutbound PR #79 October 18, 2024 15:29 — with Render Destroyed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants