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

Integration github-upload with the rest of the app #35

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

Conversation

batunpc
Copy link
Collaborator

@batunpc batunpc commented Dec 9, 2022

Fixes #33

As discussed in the issue, current behavior of the app is completely isolated from github-upload function using Octokit. Which was implemented at #11

This PR integrates the "upload images to authenticated repo using github-upload" functionality into the front-end, as it was programmatically written for client side.

Steps to reproduce

  1. First create an public repository with the desired name.

  2. Next, navigate to Settings > Developer settings > PAT > Tokens(Classic) to create a brand-new token.
    Ensure you have provided PAT permission to the created repository from step 1.

  3. After token is created you can start the app with npm run start and follow the rest of the steps. After you have uploaded an supported type image and click on upload, you will see raw directory within the repo you have created.
    It will look something like this.

@batunpc batunpc requested review from humphd and saminarp December 9, 2022 11:28
@batunpc batunpc self-assigned this Dec 9, 2022
src/App.js Outdated Show resolved Hide resolved
src/App.js Show resolved Hide resolved
@batunpc
Copy link
Collaborator Author

batunpc commented Dec 10, 2022

Thank you for your comments @humphd please let me know if I should do any other change for this PR

I have filed follow-up issue regarding your comment, and will be raising PR soon.

Copy link
Collaborator

@humphd humphd left a comment

Choose a reason for hiding this comment

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

Looks good, one minor thing.

src/App.js Outdated Show resolved Hide resolved
@batunpc
Copy link
Collaborator Author

batunpc commented Dec 16, 2022

Thank you @humphd I have implemented the suggested change 👍

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.

Integrate github-upload with the rest of the working code
3 participants