Skip to content

refactor(server, web): create shared link - #2879

Merged
alextran1502 merged 3 commits into
mainfrom
refactor/create-shared-link
Jun 21, 2023
Merged

alextran1502 merged 3 commits into
mainfrom
refactor/create-shared-link

Conversation

@jrasm91

@jrasm91 jrasm91 commented Jun 20, 2023

Copy link
Copy Markdown
Member

No description provided.

@vercel

vercel Bot commented Jun 20, 2023 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment
Name Status Preview Comments Updated (UTC)
immich ⬜️ Ignored (Inspect) Jun 20, 2023 6:42pm

@jrasm91
jrasm91 force-pushed the refactor/create-shared-link branch from a5a5074 to 1fe0417 Compare June 20, 2023 18:02
@jrasm91
jrasm91 force-pushed the refactor/create-shared-link branch from 1fe0417 to 58d90f2 Compare June 20, 2023 18:36
@jrasm91
jrasm91 requested a review from alextran1502 June 20, 2023 19:17

<section class="m-6">
{#if !isShowSharedLink}
{#if !sharedLink}

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Would a flattened chain like this make more sense?

{#if sharedLink}
{:else if editingLink}
{:else}
{/if}

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Maybe. I think there should be separate components for create vs edit myself.

@alextran1502
alextran1502 merged commit 868f629 into main Jun 21, 2023
@alextran1502
alextran1502 deleted the refactor/create-shared-link branch June 21, 2023 01:08
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.

3 participants