Skip to content
This repository was archived by the owner on Sep 30, 2024. It is now read-only.

feat(plg): remove accept invite page#63290

Merged
taras-yemets merged 2 commits into
mainfrom
ty/remove-accept-invite-page
Jun 18, 2024
Merged

feat(plg): remove accept invite page#63290
taras-yemets merged 2 commits into
mainfrom
ty/remove-accept-invite-page

Conversation

@taras-yemets

@taras-yemets taras-yemets commented Jun 17, 2024

Copy link
Copy Markdown
Contributor

Removes "/cody/invites/accept" page.

Invite processing is handled on the "/cody/manage" page and "/cody/invites/accept" is used only to redirect to the former.

Backend counterpart: https://github.com/sourcegraph/self-serve-cody/pull/886

Test plan

  • Tested manually
    • checkout branch fromhttps://github.com/sourcegraph/self-serve-cody/pull/886 and run SSC backend locally
    • run Sourcegraoh instance in dotcom mode locally
    • log in as a Cody Pro team admin and send an invite to another user
    • log in as an invited user and follow the link from the email
    • ensure link navigates to the "/cody/manage/" page and is hanled as expected (see test plan from https://github.com/sourcegraph/sourcegraph/pull/63231)

Changelog

@cla-bot cla-bot Bot added the cla-signed label Jun 17, 2024
@taras-yemets taras-yemets requested a review from a team June 17, 2024 12:49
@taras-yemets taras-yemets marked this pull request as ready for review June 17, 2024 12:49

@vdavid vdavid left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

👍

@taras-yemets taras-yemets merged commit 9b88b93 into main Jun 18, 2024
@taras-yemets taras-yemets deleted the ty/remove-accept-invite-page branch June 18, 2024 09:10
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants