Skip to content

Feat: magic link everywhere#1084

Open
winged wants to merge 3 commits intoadfinis:mainfrom
winged:feat_magic_link_everywhere
Open

Feat: magic link everywhere#1084
winged wants to merge 3 commits intoadfinis:mainfrom
winged:feat_magic_link_everywhere

Conversation

@winged
Copy link
Copy Markdown
Member

@winged winged commented Feb 18, 2026

This makes it much easier to share magic links, as we don't have to
manually enter the correct task etc, if we have an activity or task
that matches our request already.

We now also can create reports from the magic-link dialog, which
allows us a flexible workflow of "resuming" activities
or reports by clicking the magic link, then clicking "create report", which
will then behave just as if we clicked someone else's magic link

This makes it much easier to share magic links, as we don't have to
manually enter the correct task etc, if we have an activity or task
that matches our request already.
@winged winged requested a review from a team as a code owner February 18, 2026 18:20
@winged
Copy link
Copy Markdown
Member Author

winged commented Feb 18, 2026

Implements #79 (mostly)

</div>
<div
class="form-list-cell form-group cell-buttons grid grid-cols-2 justify-around gap-2 self-center text-sm [&>*]:px-2"
class="form-list-cell form-group cell-buttons grid grid-cols-3 justify-around gap-2 self-center text-sm [&>*]:px-2"
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This makes it look off

Image

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I think that needs a change in the grid layout of the wrapping ValidatedForm > lg:grid-cols-[...repeat(3,[...])]

    <ValidatedForm
      @model={{cs}}
      class="report-row grid grid-cols-4 gap-2 p-1 lg:grid-cols-[repeat(3,minmax(0,0.9fr)),minmax(0,1.6fr),minmax(0,0.45fr),minmax(2rem,0.6fr),repeat(2,minmax(2rem,0.6fr))] lg:p-1.5 xl:grid-cols-[repeat(3,minmax(0,1.2fr)),minmax(0,1.8fr),minmax(0,0.4fr),minmax(2rem,0.5fr),repeat(3,minmax(2rem,0.5fr))] xl:p-2.5 max-lg:[&>*]:w-full"
>...</ValidatedForm>

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Instead of adding the button to every row, couldn't we move it into the edit view of the report? Would lead to less overload of the UI.

Image

This allows us a flexible workflow of "resuming" activities
or reports by clicking the magic link, then clicking "create report", which
will then behave just as if we clicked someone else's magic link
@winged winged force-pushed the feat_magic_link_everywhere branch from 39ed822 to ff07682 Compare February 19, 2026 16:54
This breaks builds and dev env - let's add it again later
@winged winged force-pushed the feat_magic_link_everywhere branch from 2f1488f to 3aad90b Compare February 19, 2026 17:17
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.

2 participants