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.

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

Get project endpoint #3

Closed
wants to merge 17 commits into from
Closed

Get project endpoint #3

wants to merge 17 commits into from

Conversation

RRap0so
Copy link
Owner

@RRap0so RRap0so commented Feb 9, 2024

Tracking issue

https://github.com/flyteorg/flyte/issues/

Why are the changes needed?

What changes were proposed in this pull request?

How was this patch tested?

Setup process

Screenshots

Check all the applicable boxes

  • I updated the documentation accordingly.
  • All new and existing tests passed.
  • All commits are signed-off.

Related PRs

Docs link

RRap0so and others added 17 commits February 2, 2024 07:58
@RRap0so
Feature new getProject endpoint
2c328f6 
Signed-off-by: Rafael Raposo <[email protected]>
@RRap0so
Better cov
e3e85d8 
Signed-off-by: Rafael Raposo <[email protected]>
@RRap0so
Merge pull request #2 from RRap0so/get_project_endpoint_add_more_cov
40dde18 
Better code coverage
@RRap0so
Add new endpoint to admin proto
8114381 
Signed-off-by: Rafael Raposo <[email protected]>
@RRap0so
Move from ProjectReq
d3ec634 
Signed-off-by: Rafael Raposo <[email protected]>
@RRap0so
Regenerate
8dfccf3 
Signed-off-by: Rafael Raposo <[email protected]>
@RRap0so
Merge
eeca446 
Signed-off-by: Rafael Raposo <[email protected]>
@RRap0so
Regenerate after merge master
4a54dbe 
Signed-off-by: Rafael Raposo <[email protected]>
@RRap0so
Bring back project Get Requests
1a26fe9 
Signed-off-by: Rafael Raposo <[email protected]>
@RRap0so
Merge with main
d96d014 
Signed-off-by: Rafael Raposo <[email protected]>
@RRap0so
go tidy and compile
0656882 
Signed-off-by: Rafael Raposo <[email protected]>
@RRap0so
address comments
e9bb20a 
Signed-off-by: Rafael Raposo <[email protected]>
@RRap0so
Typo and unnecessary field
ad1ca82 
Signed-off-by: Rafael Raposo <[email protected]>
@RRap0so
Typo and unnecessary field
12caf27 
Signed-off-by: Rafael Raposo <[email protected]>
@RRap0so
Validate input and add test case
bea1a0b 
Signed-off-by: Rafael Raposo <[email protected]>
@RRap0so
nit: same assert for errors
39a6ccd 
Signed-off-by: Rafael Raposo <[email protected]>
@RRap0so
Moved Validation to it's right place.
681d2fb 
Signed-off-by: Rafael Raposo <[email protected]>
@RRap0so RRap0so closed this Mar 20, 2024
@RRap0so RRap0so deleted the get_project_endpoint branch March 20, 2024 18:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@RRap0so