Skip to content

Conversation

@longshuicy
Copy link
Member

  • Breakpoint should work ok now
  • Move dataset description to the correct container
  • Move RoleChip to the Details for both file and dataset
image

@longshuicy longshuicy linked an issue Mar 12, 2024 that may be closed by this pull request
Copy link
Contributor

@tcnichol tcnichol left a comment

Choose a reason for hiding this comment

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

This looks good. Details changes on all pages, marking approved.

Copy link
Member

@ddey2 ddey2 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

@lmarini lmarini changed the base branch from main to release/v2.0.0-beta2 March 21, 2024 14:09
longshuicy and others added 22 commits March 22, 2024 09:31
* Removed unused imports using autoflake.

* Ordered imports using isort.

* Reformatted using black.

* Setting up pre-commits.

* Ran pre-commit an all files.

* Working on creating openapi.json without running backend to be used with codegen.

* Fixed `MUI: The Tabs component doesn't accept a Fragment as a child.`

* Fixed runtime error due to replacing `== None` with 'is None'.

* Fixed runtime error due to replacing `== None` with 'is None'.

* Automatically generate openapi json file using pre-commit hook.

* Added codegen precommit hook.

* Setting up prettier and eslint.

* First run of prettier.

* Runnin eslint and prettier as hooks.

* Skip codegen directories when running eslint and prettier.

* Flake8 E711 exception. Required for Beanie query to work.

* Fixed search_users_prefix codegen stub.

* Fixed beanie query for get_dataset_folders.

* Ignore Flake8 E711 since beanie query do not support `is None`.
* basic message, need to improve

* adding a login or register button

* removing buttons

* links to login or register if no public datasets

* changing text
pycharm was complaining about missing publicView boolean in DisplayMetadata
* initial commit, add dependency, new class
will eventually combine all file uploads

* working on adding upload drag and drop

* page does not load

* drag and drop does not actually work

* addign fileinputdrop as independent component

* add fileupload drop inside uploadfiledraganddrop

* drag and drop does not add files
select selects files but misses the first one

* drag and drop works now, is overwritten if file selected using button, need to fix

* uploads but overwrites, need to fix

* works for the select, need to fix other

* should work for drop now too

* was not an array

* delete icon works
removing console logs

* drag and drop and select means other upload methods no longer needed

* removing text in drag and drop

* added to styles but not using because it does not work

* some changes, not sure how to do rest

* fix button

* return to dataset

* remove unnecessary imports

---------

Co-authored-by: Chen Wang <[email protected]>
@longshuicy longshuicy requested a review from max-zilla as a code owner March 22, 2024 14:38
@longshuicy longshuicy merged commit cc0bd83 into release/v2.0.0-beta2 Mar 22, 2024
@longshuicy longshuicy deleted the 949-details-column-on-dataset-page-whitespace branch March 22, 2024 14:55
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.

Details column on dataset page whitespace

5 participants