Skip to content

update link for LWC to point to dashboard and update CSP status from …#100

Open
Julian88Tex wants to merge 1 commit into
masterfrom
feature/jjoseph/update_link_and_update_status_name
Open

update link for LWC to point to dashboard and update CSP status from …#100
Julian88Tex wants to merge 1 commit into
masterfrom
feature/jjoseph/update_link_and_update_status_name

Conversation

@Julian88Tex

Copy link
Copy Markdown
Member

…potential to google generated

@Julian88Tex Julian88Tex requested a review from sungysang May 13, 2020 07:43
<lightning-formatted-text class="slds-p-left_medium" value="Success! "></lightning-formatted-text>
<a href={cspRelatedListUrl}>Click here to see new and existing Church School Partnerships.</a>
<br>
<a href=https://lovinghouston--partial.lightning.force.com/lightning/r/Dashboard/01Z3l000001IG0fEAG/view>Click here to see new and existing Church School Partnerships.</a>

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

@sungysang I hard coded this but by any chance would you be able to make this dashboard link dynamic?

partnership.Church_Account__c = church.Id;
partnership.School_Account__c = account;
partnership.Status__c = 'Potential';
partnership.Status__c = 'Google-Generated';

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

@sungysang lmk if you think this will cause any issues

// Raw churches will be inserted
churchesNeedCSP.addAll(GooglePlacesDML.createProcessedChurches(processedChurches.get('Non-dupes')));
List<Church_School_Partnership__c> csps = GooglePlacesDML.createPotentialCSPs(churchesNeedCSP, account);
List<Church_School_Partnership__c> csps = GooglePlacesDML.createGoogleGeneratedCSPs(churchesNeedCSP, account);

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

@sungysang lmk if you think this will cause any issues

@Julian88Tex Julian88Tex self-assigned this May 28, 2020
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.

1 participant