Skip to content

feat: Create invitation links for join to team#4887

Open
Koc wants to merge 1 commit intomainfrom
feature/create-invitation-links
Open

feat: Create invitation links for join to team#4887
Koc wants to merge 1 commit intomainfrom
feature/create-invitation-links

Conversation

@Koc
Copy link
Copy Markdown

@Koc Koc commented Nov 29, 2025

This PR adds possibility to generate invitation link (even for completely hidden/closed Circles). Any user is able to join to C circle by opening this link.

Preview

Invitation link management

image image

Invitation link opened

image

Invitation link opened when user already joined to a Circle

image

Invitation link opened when user already requested join to a Circle

image

Dependencies

Todo

  • Cover new functionality with tests after initial code review

@Koc Koc force-pushed the feature/create-invitation-links branch 5 times, most recently from 97bbf10 to d073a9b Compare November 29, 2025 22:59
@codecov
Copy link
Copy Markdown

codecov Bot commented Nov 29, 2025

Codecov Report

❌ Patch coverage is 0% with 6 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
lib/Controller/PageController.php 0.00% 5 Missing ⚠️
appinfo/routes.php 0.00% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

@Koc Koc force-pushed the feature/create-invitation-links branch 5 times, most recently from e887136 to 73eab05 Compare November 30, 2025 22:19
Comment thread templates/message.php Outdated
@Koc Koc force-pushed the feature/create-invitation-links branch 3 times, most recently from f7d5f60 to 04cef57 Compare December 7, 2025 13:20
@Koc Koc marked this pull request as ready for review December 7, 2025 13:26
Comment thread src/components/JoinInvitation.vue Outdated
Comment thread src/models/constants.ts
@GVodyanov
Copy link
Copy Markdown
Contributor

This is looking like a cool PR! I really like this feature, and the code (at least the front-end which I'm responsible for), is looking good.

However, I am having an issue where I am not able to load my previously created teams when checking out your PR in circles, is this something others are also experiencing?

@GVodyanov
Copy link
Copy Markdown
Contributor

@nfebe Hey! What do you think of this? You maintain the frontend for circles right?

@Koc Koc force-pushed the feature/create-invitation-links branch from 04cef57 to d114d0d Compare April 16, 2026 23:30
Signed-off-by: Kostiantyn Miakshyn <molodchick@gmail.com>
@Koc Koc force-pushed the feature/create-invitation-links branch from d114d0d to 7f90869 Compare April 16, 2026 23:35
@Koc
Copy link
Copy Markdown
Author

Koc commented Apr 16, 2026

@GVodyanov

However, I am having an issue where I am not able to load my previously created teams when checking out your PR in circles, is this something others are also experiencing?

I haven't anything like that. What error do you have? Please provide more detals, e.g. logs, screenshoots, etc

Rebased PR, BTW.

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