aep.dev icon indicating copy to clipboard operation
aep.dev copied to clipboard

Adopt AEP 231 - Batch Get

Open mkistler opened this issue 1 year ago • 4 comments

Adopt AEP 231 - Batch Get

🍱 Types of changes

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

  • [ ] Enhancement
  • [ ] New proposal
  • [X] Migrated from google.aip.dev
  • [ ] Chore / Quick Fix

📋 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

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

💝 Thank you!

mkistler avatar Apr 22 '24 14:04 mkistler

@rofrankel Please re-review.

mkistler avatar May 10 '24 11:05 mkistler

@rofrankel Thanks for the re-review. I addressed your comments and asked for a new review.

mkistler avatar May 17 '24 14:05 mkistler

I made the updates we discussed three weeks ago. Please re-review and let me know if there is any additional feedback.

mkistler avatar Sep 13 '24 18:09 mkistler

@rofrankel Thanks for the comments. I've addressed or answered all that I could. I will need help on the proto definition of the response.

I will tag you for re-review.

mkistler avatar Sep 27 '24 19:09 mkistler

Thank you for the review and suggestions @toumorokoshi. I accepted all these. Is there anything else to do or can we merge this now?

mkistler avatar Oct 25 '24 15:10 mkistler