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

[W-8857319] SFDX conversion and dataset updates #74

Merged
merged 5 commits into from
Feb 11, 2021

Conversation

davidmreed
Copy link
Contributor

This PR captures maintenance work done for a demo, including converting Outbound Funds to SFDX format and updating the dataset and mapping file.

Critical Changes

Changes

Issues Closed

@davidmreed davidmreed requested a review from a team as a code owner November 11, 2020 19:17
@spelak-salesforce spelak-salesforce added the 🎯 this sprint This Pull Request is related to a Work Item committed in this sprint label Feb 5, 2021
@spelak-salesforce
Copy link
Contributor

@davidmreed , FYI: Orion will is planning on merging this PR this sprint. We are in progress creating a child branch to fix the Label Audit errors. We will dev-review this PR, and we will need a RelEng review after merging our Label Audit fix.

@spelak-salesforce spelak-salesforce changed the title SFDX conversion and dataset updates [W-8857319] SFDX conversion and dataset updates Feb 5, 2021
@spelak-salesforce
Copy link
Contributor

W-8857319

@spelak-salesforce spelak-salesforce added the 🚧 wip 🧯💨 Work in progress label Feb 5, 2021
…dit to pass (#83)

* Updating Hardcoded string to Constants on Disbursement Helper JS File.

* Removing cons logs and missing hardcoded strings

* Removing setDefaultModel call Disbursement Controller file.
setDefaultModel is now being called from the init method in helper file.

* Removing extra spaces under attributes definition

* Remove defaultModel attribute usage.
- The DEFAULT_MODEL values are set directly on setDefaultModel method so
  there is not need to use an aura attribute to perform this operation.
@spelak-salesforce spelak-salesforce added the needs for dev review Ready for Developer review label Feb 9, 2021
@spelak-salesforce
Copy link
Contributor

@davidmreed , can you give this a look over before we start reviewing. Also, can we get a code owner review by @SalesforceFoundation/release-engineering-reviewers ? Devs will start reviewing.

@davidmreed
Copy link
Contributor Author

@spelak-salesforce I created this PR but I think it's ready to go. @jstvz will have to approve it for CODEOWNERS since I cannot.

@davidmreed davidmreed requested a review from jstvz February 9, 2021 21:40
Copy link
Member

@jstvz jstvz left a comment

Choose a reason for hiding this comment

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

CODEOWNERS 🆗

@spelak-salesforce spelak-salesforce removed the 🚧 wip 🧯💨 Work in progress label Feb 10, 2021
@scottwarren-sfdo scottwarren-sfdo added ready for QE This item is ready for testing and removed needs for dev review Ready for Developer review labels Feb 10, 2021
@gaganpsandhu gaganpsandhu added QE in Progress QE is actively reviewing this Pull Request and removed ready for QE This item is ready for testing labels Feb 10, 2021
@scottwarren-sfdo scottwarren-sfdo added the Parent Branch The PR is associated with a parent branch upon which others depend label Feb 10, 2021
@gaganpsandhu gaganpsandhu merged commit 1c0f27c into master Feb 11, 2021
@gaganpsandhu gaganpsandhu deleted the feature/sfdx-and-dataset-updates branch February 11, 2021 20:28
@gaganpsandhu gaganpsandhu removed the QE in Progress QE is actively reviewing this Pull Request label Feb 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Parent Branch The PR is associated with a parent branch upon which others depend 🎯 this sprint This Pull Request is related to a Work Item committed in this sprint
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants