Skip to content
This repository was archived by the owner on Sep 3, 2022. It is now read-only.

Replace @ndhoule/pick with lodash.pick #210

Merged
merged 2 commits into from
Sep 16, 2020
Merged

Conversation

bryanmikaelian
Copy link
Contributor

@bryanmikaelian bryanmikaelian commented Sep 16, 2020

Description

This PR replaces the function pick, provided by @ndhoule/pick, with the version provided by lodash.pick

Test plan

Testing completed successfully using unit tests and e2e tests. Note how my prop search overrides the context.page value.

Screen Shot 2020-09-16 at 3 06 55 PM

Checklist

  • Thorough explanation of the issue/solution, and a link to the related issue
  • CI tests are passing
  • Unit tests were written for any new code
  • Code coverage is at least maintained, or increased.

@bryanmikaelian bryanmikaelian marked this pull request as ready for review September 16, 2020 18:54
@bryanmikaelian bryanmikaelian requested a review from a team September 16, 2020 18:54
@bryanmikaelian bryanmikaelian merged commit 6e33d3d into master Sep 16, 2020
@bryanmikaelian bryanmikaelian deleted the bryan/replace-pick branch September 16, 2020 19:13
juliofarah added a commit that referenced this pull request Mar 19, 2021
hbrls pushed a commit to nice-fungal/analytics.js-core that referenced this pull request Apr 16, 2021
hbrls pushed a commit to nice-fungal/analytics.js-core that referenced this pull request Apr 16, 2021
hbrls pushed a commit to nice-fungal/analytics.js-core that referenced this pull request Apr 16, 2021
hbrls pushed a commit to nice-fungal/analytics.js-core that referenced this pull request Apr 16, 2021
hbrls pushed a commit to nice-fungal/analytics.js-core that referenced this pull request May 31, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants