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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Reading across collections globally #165

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

rofrankel
Copy link
Collaborator

馃搼 Proposed changes

Add guidance for supporting reading across collections with different resource hierarchies.

Note: this is downstream of #164.

馃嵄 Types of changes

What types of changes does your code introduce to AEP? Put an x in the boxes
that apply

馃搵 Your checklist for this pull request

Please review the AEP Style and Guidance for
contributing to this repository.

General

Additional checklist for a new AEP

  • A new AEP should be no more than two pages if printed out.
  • Ensure that the PR is editable by maintainers.
  • Ensure that File structure
    guidelines are met.
  • Ensure that
    Document structure
    guidelines are met.

馃挐 Thank you!

@rofrankel rofrankel requested a review from a team as a code owner April 10, 2024 21:00
@toumorokoshi toumorokoshi self-requested a review April 12, 2024 19:21
Copy link
Member

@toumorokoshi toumorokoshi left a comment

Choose a reason for hiding this comment

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

initial review done! a couple comments to start a discussion.

aep/general/0159/aep.md.j2 Outdated Show resolved Hide resolved
aep/general/0159/aep.md.j2 Outdated Show resolved Hide resolved
@rofrankel rofrankel mentioned this pull request Apr 25, 2024
9 tasks
@rofrankel rofrankel force-pushed the reading-across-collections-globally branch from 9dbb8cf to 98a35c6 Compare May 31, 2024 23:38
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.

None yet

3 participants