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

feat: Import public GitHub repository #8

Merged
merged 26 commits into from
Mar 22, 2024
Merged

feat: Import public GitHub repository #8

merged 26 commits into from
Mar 22, 2024

Conversation

DiggidyDev
Copy link
Owner

Description:

Ability to import a project structure from GitHub

Changes Made:

  • Input field for GitHub URL
    • Only supports public repos (for now)
    • See README for valid URLs
  • Refactored main page into multiple components
    • <BottomPanel />
    • <OptionsPanel />
    • <ValidationForm />
    • <ActionInput />
    • <Option />
      • Empty for now, but should be utilised soon
  • Using context to pass data between components

Related Issues/Pull Requests:

#3

Testing notes (+ instructions):

New test commands:

  • npm run test
  • npm run test:e2e
  • npm run test:cmp

Added E2E tests and component tests to pipeline

Screenshots/GIFs:

N/A

Performance Impact:

Pipeline runtime has increased slightly due to new unit tests, along with code coverage upload

Dependencies:

Documentation Updates:

README has documented new test commands, and new endpoint /api/files

Additional Notes:

N/A

Reviewer Checklist:

  • Code quality and style
  • Functionality and behaviour
  • Edge cases and error handling
  • Performance considerations
  • Documentation completeness
  • Testing coverage and results

@DiggidyDev
Remove placeholder endpoint URL from /api/validate doc
51e28f3
@DiggidyDev
Install octokit/request for backend calls to GitHub
bc31d81
@DiggidyDev
Add styling for component tests
7e00db4
@DiggidyDev
Move CSS import from commands.ts -> component.ts
0fe70a0
@DiggidyDev
Rejig home page to center button
dd47bf9
@DiggidyDev
Validation utility for extracting owner, repository, and tree from Gi…
0628a72 
…tHub URL
@DiggidyDev
Set max screen width for content
828fb12
@DiggidyDev
- Added options menu toggle button
75169ec 
- Edited e2e tests to account for options toggle menu
- New Button Component
@DiggidyDev
Move options to prevent buttons from jumping about the page
e7b383b
@DiggidyDev
Initial setup for /api/files endpoint (to retrieve github repo direct…
6e7a697 
…ory structures)
@DiggidyDev
Add functionality to "Import from GitHub" button
cb600f9
@DiggidyDev
refactor postcss -> commonjs
f37eb6a
@DiggidyDev
Minimum width for show/hide options button to not disturb layout of b…
fc72c5f 
…ottom buttons
@DiggidyDev
Initial dockerignore auto-population
385cf3a
@DiggidyDev
Added schema to vercel.json file for intellisense
c1e6146
@DiggidyDev
refactored isFetchingRepo -> isFetchingFiles
0aedb59
@DiggidyDev
- added optional www. to regex
70ef991 
- made regex's https://www. entirely optional
@DiggidyDev
Initial mocha unit tests + code coverage
506aaa0
@DiggidyDev
Remove unused babel nyc plugin
ca302b1
@DiggidyDev
Update deployment and e2e deps
2ff5aa9
@DiggidyDev
- Added clover reporter to nyc
f6a99d7 
- Initial code coverage github action
@DiggidyDev
- updated sample repo url
e3a8051 
- removed blob from url validation
- updated validation tests to test entire regex
@DiggidyDev
- Moved octoFetch to separate serverless api utils file
96182e5 
- New /api/files params to control response content
- /api/files param validation
- Updated readme to reflect changes
@DiggidyDev
- Fixed main content layout for larger screens (space evenly)
272f1b4 
- Button and TextArea dark/light theme support for states
@DiggidyDev
- api utils unit test
a00445c 
- Add sinon for fetch() stubbing
@DiggidyDev
Major refactor for separation of concerns
0471b9e
@DiggidyDev DiggidyDev added the enhancement New feature or request label Mar 22, 2024
@DiggidyDev DiggidyDev self-assigned this Mar 22, 2024
@DiggidyDev DiggidyDev linked an issue Mar 22, 2024 that may be closed by this pull request
[REQ] URL Input to accept public GitHub repos #6
Closed
@DiggidyDev DiggidyDev changed the title [FEAT] Import GitHub Repository feat: Import public GitHub repository Mar 22, 2024
@DiggidyDev DiggidyDev merged commit 0e38c02 into main Mar 22, 2024
8 checks passed
@DiggidyDev DiggidyDev deleted the feature/import-repo branch March 22, 2024 22:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@DiggidyDev DiggidyDev

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[REQ] URL Input to accept public GitHub repos
1 participant
@DiggidyDev