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.

Sign up for GitHub

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

Jump to bottom

Invite email multi site #579

Merged
merged 7 commits into from
Feb 21, 2023
Merged

Invite email multi site #579

merged 7 commits into from
Feb 21, 2023

Conversation

MyPyDavid
Copy link
Member

@MyPyDavid MyPyDavid commented Feb 3, 2023 •
edited
Loading

For #576

  • add the tests

@MyPyDavid MyPyDavid self-assigned this Feb 3, 2023
@MyPyDavid MyPyDavid linked an issue Feb 3, 2023 that may be closed by this pull request
Handling of project invitation links across multi sites #576
Closed
@johlton johlton mentioned this pull request Feb 3, 2023
Closed
@MyPyDavid MyPyDavid requested a review from jochenklar February 15, 2023 16:27
@MyPyDavid MyPyDavid added this to the 1.9.2 milestone Feb 16, 2023
jochenklar
jochenklar approved these changes Feb 21, 2023
View reviewed changes
rdmo/projects/utils.py Outdated Show resolved Hide resolved
@jochenklar
Copy link
Member

All fine, can be merged. Is it still WIP, @MyPyDavid ?

@MyPyDavid
Copy link
Member Author

no after your review not anymore ;) thanks @jochenklar

jochenklar
jochenklar reviewed Feb 21, 2023
View reviewed changes
rdmo/projects/utils.py Outdated Show resolved Hide resolved
@jochenklar jochenklar marked this pull request as ready for review February 21, 2023 14:17
@jochenklar jochenklar changed the base branch from dev-1.9.1 to dev February 21, 2023 14:18
@jochenklar jochenklar merged commit 28809d6 into dev Feb 21, 2023
@jochenklar jochenklar deleted the draft/invite-email-multi-site branch February 21, 2023 14:19
@coveralls
Copy link

Coverage Status

Coverage: 90.462% (+0.03%) from 90.433% when pulling 6fbf861 on draft/invite-email-multi-site into 9d79533 on dev.

CalamityC pushed a commit to CalamityC/rdmo that referenced this pull request Nov 23, 2023
@jochenklar
Merge pull request rdmorganiser#579 from rdmorganiser/draft/invite-em…
7c2e3c9 
…ail-multi-site

Invite email multi site
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jochenklar jochenklar jochenklar approved these changes

Assignees

@MyPyDavid MyPyDavid

Labels
type:feature-request
Projects
None yet
Milestone
1.9.2
Development

Successfully merging this pull request may close these issues.

Handling of project invitation links across multi sites
3 participants
@MyPyDavid @jochenklar @coveralls