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

Migrate to SFDX source format #259

Closed
afawcett opened this issue Dec 28, 2019 · 1 comment · Fixed by #274
Closed

Migrate to SFDX source format #259

afawcett opened this issue Dec 28, 2019 · 1 comment · Fixed by #274

Comments

@afawcett
Copy link
Contributor

afawcett commented Dec 28, 2019

And thus also supporting libraries such as Mocks and Sample repos.

NOTE: This will also need an update to the either one of the Deploy to Salesforce buttons to continue to support deploying to non-scratch orgs.

@afawcett
Copy link
Contributor Author

Per a comment I left on one of the PR's I am thinking while we need to get this done, it may serve well in the interests of getting some of the longer standing PR's merge to wait a little longer to avoid creating merge issues with those PR's. Thoughts?

@ImJohnMDaniel ImJohnMDaniel self-assigned this Dec 30, 2019
ImJohnMDaniel added a commit that referenced this issue Apr 5, 2020
Changes made:
* Migrated all code to Salesforce DX Source format
@ImJohnMDaniel ImJohnMDaniel moved this from To do to In progress in Migrate all projects to SFDX Source Format Apr 6, 2020
@ImJohnMDaniel ImJohnMDaniel linked a pull request Apr 9, 2020 that will close this issue
Migrate all projects to SFDX Source Format automation moved this from In progress to Done Apr 13, 2020
ImJohnMDaniel added a commit that referenced this issue Apr 13, 2020
…on-to-sfdx-format

Fixes #259 -- Migration of code to SFDX Source format
jsardello pushed a commit to lampo/salesforce-apex-common that referenced this issue Jul 10, 2020
* Added new overload to registerRelationship to allow reference by External Id. The changes cannot be covered by a Unit Test due to lack of such references on standard sObjects. Changes were tested manually in a scratch org in execute anonymous using custom objects.

* Default Sorting considering invalid field and shield

Financial Force library (FFLIB) by default sorts record based on CreatedDate. Some object (ie. account share) might not have the field and hence will through error.  This change checks the field encryption (taken from @wimvelzeboer ) and if the object does not have created date field, assigns Id as default sorting field.

* Cleaned up the code based on pull request feedback.

* Update fflib_SObjectSelector.cls

Fixing the comment-blocking indenting issues (caused by IDE)

* Update fflib_SObjectSelector.cls

Updated the access modifiers from protected to private.

* correct spelling of word updates within comment and exception message

* Adding new method to mocks

* Rebrand as FFLib Apex Common & correct copyright notices to match git history

* ChildRelationship -> Schema.ChildRelationship

* upgrade api version

* Allow mocking for getChangedRecords

The hardcoded reference to the readonly Trigger.oldMap is replace so that it can be mocked in unit-tests

* Add mocking for getChangedRecords via Test.Database

* Add empty recycle bin to UnitOfWork

* Update README.md

* Add fluent design to the class constructor

With this change a selector can be configured via AccountsSelector selector = new AccountsSelector().enforceFLS();

* Add missing return statement for fluent design to the class constructor

With this change a selector can be configured via AccountsSelector selector = new AccountsSelector().enforceFLS();

* Add the option to extend the Application class.

This will allow an extension that could combine the force-di repo with the fflib-apex-commons.

* Updated to 47.0

* Fix for failing unit test by adding required methods to implementations

* 186 - consistent license version across source

* apex-enterprise-patterns#199 - mark fflib_SObjectSelector methods as virtual that do actual SOQL -- permits override in a 'without sharing' subclass

* Fixes apex-enterprise-patterns#263 - test assertion using string with typo in it (caused by apex-enterprise-patterns#234)

* Update fflib_SObjectUnitOfWork.cls

typo

* Fixes apex-enterprise-patterns#259

Changes made:
* Migrated all code to Salesforce DX Source format

* Update to document this change

* Update README.md

Updated Deploy button to not hard code the repo name. This means branch deploys work.

* Updating API version of Apex Commons (#1)

* Updating API version of Apex Commons

Updating API version of Apex Commons

* Adding Code Owners

Adding Code Owners

* Update from apex enterprise patterns master of fflib_Apex_Common (#3)

* Added new overload to registerRelationship to allow reference by External Id. The changes cannot be covered by a Unit Test due to lack of such references on standard sObjects. Changes were tested manually in a scratch org in execute anonymous using custom objects.

* Default Sorting considering invalid field and shield

Financial Force library (FFLIB) by default sorts record based on CreatedDate. Some object (ie. account share) might not have the field and hence will through error.  This change checks the field encryption (taken from @wimvelzeboer ) and if the object does not have created date field, assigns Id as default sorting field.

* Cleaned up the code based on pull request feedback.

* Update fflib_SObjectSelector.cls

Fixing the comment-blocking indenting issues (caused by IDE)

* Update fflib_SObjectSelector.cls

Updated the access modifiers from protected to private.

* Adding new method to mocks

* Rebrand as FFLib Apex Common & correct copyright notices to match git history

* upgrade api version

* Allow mocking for getChangedRecords

The hardcoded reference to the readonly Trigger.oldMap is replace so that it can be mocked in unit-tests

* Add mocking for getChangedRecords via Test.Database

* Update README.md

* Add fluent design to the class constructor

With this change a selector can be configured via AccountsSelector selector = new AccountsSelector().enforceFLS();

* Add missing return statement for fluent design to the class constructor

With this change a selector can be configured via AccountsSelector selector = new AccountsSelector().enforceFLS();

* Add the option to extend the Application class.

This will allow an extension that could combine the force-di repo with the fflib-apex-commons.

* Updated to 47.0

Co-authored-by: Rick Parker <45234060+TheRickParker@users.noreply.github.com>
Co-authored-by: Tahsin Zulkarnine <tahsinz@gmail.com>
Co-authored-by: dbtavernerffdc <dbtaverner@financialforce.com>
Co-authored-by: kjonesffdc <31822118+kjonesffdc@users.noreply.github.com>
Co-authored-by: Jan Aertgeerts <jan@janaertgeerts.be>
Co-authored-by: William Velzeboer <wgj.velzeboer@gmail.com>
Co-authored-by: John M. Daniel <ImJohnMDaniel@users.noreply.github.com>
Co-authored-by: Andrew Fawcett <andy@andyinthecloud.com>

* Add idea to git ignore

Co-authored-by: Rick Parker <rick.lee.parker@gmail.com>
Co-authored-by: Tahsin Zulkarnine <tahsinz@gmail.com>
Co-authored-by: dbtavernerffdc <dbtaverner@financialforce.com>
Co-authored-by: Connor Jerow <cgjerow@gmail.com>
Co-authored-by: Kevin Jones <kjones@financialforce.com>
Co-authored-by: Stefano Pallicca <stefano.pallicca@imagicle.com>
Co-authored-by: Jan Aertgeerts <jaertgee@its.jnj.com>
Co-authored-by: William Velzeboer <wgj.velzeboer@gmail.com>
Co-authored-by: William Velzeboer <wvelzebo@its.jnj.com>
Co-authored-by: Wim Velzeboer <wimvelzeboer@outlook.com>
Co-authored-by: John M. Daniel <ImJohnMDaniel@users.noreply.github.com>
Co-authored-by: Andrew Fawcett <andy@andyinthecloud.com>
Co-authored-by: David Esposito <dave@patrontechnology.com>
Co-authored-by: David Esposito <233309+daveespo@users.noreply.github.com>
Co-authored-by: Marek Madejski <marekmadejski@yandex.com>
Co-authored-by: John M. Daniel <imjohnmdaniel@ce-v.com>
Co-authored-by: Rick Parker <45234060+TheRickParker@users.noreply.github.com>
Co-authored-by: kjonesffdc <31822118+kjonesffdc@users.noreply.github.com>
Co-authored-by: Jan Aertgeerts <jan@janaertgeerts.be>
@daveespo daveespo unpinned this issue Jul 30, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Development

Successfully merging a pull request may close this issue.

2 participants