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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add API Endpoints for GitHub integration for app creation, repository, and branch management #3922

Closed
wants to merge 2 commits into from

Conversation

xmahbub
Copy link

@xmahbub xmahbub commented Oct 15, 2024

Add GitHub controller to manage GitHub apps, repositories, and branches

  • New routes for GitHub app management:
    • POST /github-apps: Create a new GitHub app
    • GET /github-apps/{github_app_id}/repositories: Load all repositories for a specific GitHub app
    • GET /github-apps/{github_app_id}/repositories/{owner}/{repo}/branches: Load all branches for a specific repository
  • Applied IgnoreReadOnlyApiToken middleware to all routes

This feature enables users to create GitHub apps, view repositories, and access repository branches.

@peaklabs-dev peaklabs-dev added the 🛠️ Feature Issues requesting a new feature. label Oct 15, 2024
@peaklabs-dev peaklabs-dev changed the title Add GitHub integration for app creation, repository, and branch management Add API Endpoints for GitHub integration for app creation, repository, and branch management Oct 15, 2024
@peaklabs-dev
Copy link
Member

This PR is the same as #3921 I think you have selected the worng branch.

@peaklabs-dev peaklabs-dev added the 💤 Waiting for feedback Issues awaiting a response from the author. label Oct 17, 2024
@xmahbub
Copy link
Author

xmahbub commented Oct 17, 2024

Yes 🫥 I was planed to seperately push this two changes from my to separate branch. Thank you for your concern.

@peaklabs-dev
Copy link
Member

So both changes are now in one PR right?

@xmahbub
Copy link
Author

xmahbub commented Oct 17, 2024

I want to have to seperate PR

@peaklabs-dev
Copy link
Member

Yes, I understand that, but right now both changes you made are on the same branch, right? You can revert the commits you made to that PR from the main branch, then move the changes to another branch, and then open a new PR for that change.

@xmahbub
Copy link
Author

xmahbub commented Oct 17, 2024

yes both are in one #3921 . Ok thank you

@xmahbub xmahbub closed this Oct 17, 2024
@github-actions github-actions bot removed 💤 Waiting for feedback Issues awaiting a response from the author. 🛠️ Feature Issues requesting a new feature. labels Oct 17, 2024
@xmahbub
Copy link
Author

xmahbub commented Oct 17, 2024

I have closed this PR and change the title of the fist PR #3921 and also change the details comment. Is it OK now

@peaklabs-dev
Copy link
Member

Perfect thanks

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 17, 2024
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