Skip to content

Conversation

@ddey2
Copy link
Member

@ddey2 ddey2 commented Jun 20, 2024

  1. Make sure you have no metadata defn for files
  2. Try to upload a file
  3. You should not see the step1 saying "Fill in metadata"

Current UI:
Screenshot 2024-06-20 at 11 36 45 AM

@ddey2 ddey2 requested a review from longshuicy as a code owner June 20, 2024 16:36
@ddey2 ddey2 linked an issue Jun 20, 2024 that may be closed by this pull request
@ddey2 ddey2 requested review from Vismayak, lmarini and tcnichol June 20, 2024 16:37
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. With no metadata definitions, this does not appear for files.

… has a required field and also hide back button if there is no step1
@ddey2 ddey2 requested a review from longshuicy June 27, 2024 21:11
@lmarini lmarini added this to the v2.0-beta-3 milestone Jun 28, 2024
Copy link
Member

@longshuicy longshuicy left a comment

Choose a reason for hiding this comment

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

Could you check now even if metadata is required it's not showing the first step. See screenshot:

image

@ddey2
Copy link
Member Author

ddey2 commented Jun 29, 2024

@longshuicy It will not show up as the required metadata has no required field.

@longshuicy
Copy link
Member

@longshuicy It will not show up as the required metadata has no required field.

Ah my bad. I understand now. Approved.

@longshuicy longshuicy merged commit 07e22ea into main Jul 1, 2024
@longshuicy longshuicy deleted the 1035-file-upload-pop-up-window branch July 1, 2024 14:29
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.

File upload pop up window

5 participants