-
Notifications
You must be signed in to change notification settings - Fork 54
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
Basic listing of files in UI #391
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
danactive
force-pushed
the
resize/move-react-from-api-to-ui
branch
from
February 12, 2020 05:06
b8f417d
to
a21edd5
Compare
* master: Link to backend chore(CI): Better Code Hub should ignore React code as not supported chore(Explore Video): Delete failing tests feat(Explore Video): Update components from Node.js to browser code feat(Explore Video): Move YouTube player components from API to UI API or UI chore(deps): bump lodash from 4.17.11 to 4.17.13 in /ui fix: api/package.json & api/package-lock.json to reduce vulnerabilities # Conflicts: # api/webpack.js # ui/app/containers/App/index.jsx
danactive
force-pushed
the
resize/move-react-from-api-to-ui
branch
2 times, most recently
from
February 12, 2020 05:20
bb5ede7
to
e391fa8
Compare
Working towards #232 |
danactive
force-pushed
the
resize/move-react-from-api-to-ui
branch
from
February 24, 2020 04:48
e391fa8
to
0b1b277
Compare
* master: chore(Resize): Correct merge conflict feat(Docker): Add Graphicsmagick chore(Resize): Improve error message when resized images is missing feat(Album View): Update actions to avoid state; easier for debugging feat(Album View): Add Local host to view album feat(Gallery View): Add Local host to view galleries feat(API Server): All routes enable CORS to allow UI to HTTP Request cross domains chore(package): Update Dropbox with minor bump and remove warnings chore(Gallery): Resolve PR checks feat(Album View): Add Dropbox host to address feat(Gallery View): Add Dropbox host to address feat(Gallery): Add Dropbox host to address fix(Home): Display missing Dropbox access token error chore(Explore Video): Upgrade to React Hooks fix(Resize): Improve error message when GraphicsMagick is missing Needs YouTube API key # Conflicts: # .bettercodehub.yml # ui/app/containers/App/index.jsx # ui/app/containers/App/tests/__snapshots__/index.test.jsx.snap # ui/app/utils/request.js
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
Added admin landing page (accessible by URL only). Basic navigation to the walk page where clicking on the folders will reload the page to display the listing of directory file names
Test plan
No unit tests