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

Invite landing page shows input form when an invite is invalid #24486

Open
iansltx opened this issue Dec 6, 2024 · 3 comments
Open

Invite landing page shows input form when an invite is invalid #24486

iansltx opened this issue Dec 6, 2024 · 3 comments
Labels
bug Something isn't working as documented ~frontend Frontend-related issue. #g-endpoint-ops Endpoint ops product group #g-orchestration Orchestration product group :release Ready to write code. Scheduled in a release. See "Making changes" in handbook. ~released bug This bug was found in a stable release.

Comments

@iansltx
Copy link
Member

iansltx commented Dec 6, 2024

Fleet version: <= 4.61


💥  Actual behavior

Users clicking an invalid/expired invite link are presented with an invite form that looks like they can enter details and submit the form, and they don't get feedback that the invite is invalid until they submit the form.

🧑‍💻  Steps to reproduce

While not logged in (e.g. in Incognito), go to https://dogfood.fleetdm.com/login/invites/foobar

🛠️ To fix

From @noahtalerman: Please bring a live demo of the solution to the next #g-endpoint-ops design review. That way, we can review the error message copy live on the call.

On-load, spin while calling the verify invite endpoint. Show an error message if the invite isn't valid, and use the retrieved invite details to populate form fields rather than looking at query string parameters on the invite URL (we should also get rid of those query string parameters as they're duplicative).

This spin-while-verifying behavior just got implemented as part of the MFA verify page in #22078, so that should make this a little easier to implement, and the above endpoint already exists. Honestly I'm a bit surprised how much info we hand back to an unauthenticated user that has the invite token, but given that the endpoint exists we might as well use it.

@iansltx iansltx added #g-endpoint-ops Endpoint ops product group :frontend :product Product Design department (shows up on 🦢 Drafting board) bug Something isn't working as documented labels Dec 6, 2024
@lukeheath
Copy link
Member

Good UX catch!

@noahtalerman noahtalerman self-assigned this Dec 9, 2024
@lukeheath lukeheath added the ~released bug This bug was found in a stable release. label Dec 13, 2024
@noahtalerman
Copy link
Member

noahtalerman commented Dec 18, 2024

On-load, spin while calling the verify invite endpoint. Show an error message if the invite isn't valid, and use the retrieved invite details to populate form fields rather than looking at query string parameters on the invite URL (we should also get rid of those query string parameters as they're duplicative).

@iansltx this proposed solution sounds like a good one.

I'm sending this bug back to the release board.

@sharon-fdm can you ask the engineer who implements to please bring a live demo of the solution to the #g-endpoint-ops design review? That way, we can review the error message copy live on the call.

@noahtalerman noahtalerman added :release Ready to write code. Scheduled in a release. See "Making changes" in handbook. and removed :product Product Design department (shows up on 🦢 Drafting board) labels Dec 18, 2024
@noahtalerman noahtalerman removed their assignment Dec 18, 2024
@sharon-fdm sharon-fdm added ~frontend Frontend-related issue. and removed :frontend labels Dec 18, 2024
@sharon-fdm
Copy link
Collaborator

Hey team! Please add your planning poker estimate with Zenhub @jacobshandling @RachelElysia

@lukeheath lukeheath added the #g-orchestration Orchestration product group label Dec 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working as documented ~frontend Frontend-related issue. #g-endpoint-ops Endpoint ops product group #g-orchestration Orchestration product group :release Ready to write code. Scheduled in a release. See "Making changes" in handbook. ~released bug This bug was found in a stable release.
Development

No branches or pull requests

4 participants