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

Upgrade Import Modal #1008

Merged
merged 26 commits into from
Jul 15, 2024
Merged

Upgrade Import Modal #1008

merged 26 commits into from
Jul 15, 2024

Conversation

PepperLola
Copy link
Member

@PepperLola PepperLola commented Jul 1, 2024

Description

Show cached and assetpack files in the Import Mira modal, and have an option to delete the individual cached assets. Improve the APS hub/project/file management ui and add a button to sign in if the user isn't already.

Objectives

  • Show cached and assetpack files
  • Improve UI
  • Show all mira files in all hubs
  • Show sign-in button
  • Make Import Panel the main spawning panel

Known issues to solve

  • 401 error when fetching from APS href.

Known issues to ignore for now

  • Default and APS selections not properly filtering out/updating to cached items 1753

Important

Depends on #992 and #982

JIRA Issue

@PepperLola PepperLola self-assigned this Jul 1, 2024
@HunterBarclay
Copy link
Member

@PepperLola Appears to handle the BIM360 hubs now. UI is still a mess tho

@HunterBarclay
Copy link
Member

HunterBarclay commented Jul 3, 2024

@PepperLola Instead of navigating the hubs, you might want to just search for .mira files in all the projects you can access. The documentation has some examples of applying filters to grabbing files.

@PepperLola PepperLola marked this pull request as ready for review July 9, 2024 01:47
@PepperLola PepperLola requested review from HunterBarclay and a team as code owners July 9, 2024 01:47
@PepperLola PepperLola requested review from Dhruv-0-Arora and removed request for a team July 9, 2024 01:47
@PepperLola PepperLola requested a review from a team as a code owner July 9, 2024 23:01
@PepperLola PepperLola requested review from a-crowell and LucaHaverty and removed request for a team July 9, 2024 23:01
Copy link
Collaborator

@Dhruv-0-Arora Dhruv-0-Arora 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 👍

fission/src/ui/modals/mirabuf/ImportMirabufModal.tsx Outdated Show resolved Hide resolved
fission/src/ui/modals/mirabuf/ImportMirabufModal.tsx Outdated Show resolved Hide resolved
fission/src/ui/modals/mirabuf/ImportMirabufModal.tsx Outdated Show resolved Hide resolved
fission/src/ui/modals/mirabuf/ImportMirabufModal.tsx Outdated Show resolved Hide resolved
Copy link
Member

@HunterBarclay HunterBarclay left a comment

Choose a reason for hiding this comment

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

A handful of things to change

fission/src/ui/modals/mirabuf/ImportMirabufModal.tsx Outdated Show resolved Hide resolved
fission/src/ui/modals/mirabuf/ImportMirabufModal.tsx Outdated Show resolved Hide resolved
fission/src/ui/modals/mirabuf/ImportMirabufModal.tsx Outdated Show resolved Hide resolved
fission/src/ui/modals/mirabuf/ImportMirabufModal.tsx Outdated Show resolved Hide resolved
fission/src/ui/modals/mirabuf/ImportMirabufModal.tsx Outdated Show resolved Hide resolved
Copy link
Member

@BrandonPacewic BrandonPacewic left a comment

Choose a reason for hiding this comment

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

@HunterBarclay This ready for review now?

@HunterBarclay
Copy link
Member

@HunterBarclay This ready for review now?

Finally, yes

@BrandonPacewic BrandonPacewic requested review from BrandonPacewic and removed request for LucaHaverty July 12, 2024 22:55
@HunterBarclay
Copy link
Member

Trying to fix whatever the formatter is complaining about

@BrandonPacewic
Copy link
Member

Trying to fix whatever the formatter is complaining about

🫡

HunterBarclay
HunterBarclay previously approved these changes Jul 12, 2024
@HunterBarclay HunterBarclay self-requested a review July 12, 2024 23:02
@HunterBarclay HunterBarclay dismissed their stale review July 12, 2024 23:02

Now an author of PR

@HunterBarclay HunterBarclay self-assigned this Jul 12, 2024
@HunterBarclay HunterBarclay removed their request for review July 12, 2024 23:03
Comment on lines 1 to -10
/// <reference types="vite/client" />

interface ImportMetaEnv {
readonly VITE_SYNTHESIS_SERVER_PATH: string
// more env variables...
}

interface ImportMeta {
readonly env: ImportMetaEnv
}
Copy link
Member

Choose a reason for hiding this comment

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

Do we still need to save this file?

Copy link
Member

Choose a reason for hiding this comment

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

It was kept in prior to me using and we might use it in the future for other vite environment stuff, so I decided to keep it in for now.

@HunterBarclay HunterBarclay merged commit b2a5a06 into dev Jul 15, 2024
12 checks passed
@HunterBarclay HunterBarclay deleted the jwrigh/1732/upgrade-import-modal branch July 15, 2024 06:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
aps ui/ux Relating to user interface, or in general, user experience
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants