Skip to content

feat: add getObjectInfos to allowed usage list #33

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Nov 23, 2020
Merged

feat: add getObjectInfos to allowed usage list #33

merged 1 commit into from
Nov 23, 2020

Conversation

@yungcheng
Copy link
Contributor

@yungcheng yungcheng commented Jun 15, 2020

No description provided.

@kevinv11n
Copy link

kevinv11n commented Jun 15, 2020

@apapko @Gr8Gatsby what's the branching strategy for this repo for features coming in the next major release? This PR is applicable only to the 228 / Winter 21 release.

@pmdartus
Copy link
Contributor

pmdartus commented Nov 20, 2020

@yungcheng @kevinv11n We have not enabled this on the server for 228 and it won't also be enabled for 230. It should be fine merging this right now.

Is this PR still valid? I am preparing a new release of this package.

@yungcheng
Copy link
Contributor Author

yungcheng commented Nov 20, 2020 •
edited
Loading

Is this PR still valid? I am preparing a new release of this package.

It is still valid and would be great if you can merge it

@kevinv11n
Copy link

kevinv11n commented Nov 20, 2020 •
edited
Loading

We have not enabled this on the server for 228 and it won't also be enabled for 230. It should be fine merging this right now.

These eslint rules are provided to customers, right?

Assuming so this change should've gone out in 228 when getObjectInfos was released to customers.

@pmdartus pmdartus force-pushed the getObjectInfos branch 2 times, most recently from c2c83a1 to c6cac37 Compare November 23, 2020 09:17
@pmdartus pmdartus merged commit b94f28b into salesforce:master Nov 23, 2020
@pmdartus
Copy link
Contributor

pmdartus commented Nov 23, 2020

It has been published with the V0.8.0 release.

This was referenced Mar 15, 2021
Closed
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@kevinv11n kevinv11n kevinv11n approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@yungcheng @kevinv11n @pmdartus