fix: allow adding self to shared event#6418
Conversation
SebastianKrupinski
requested review from
miaulalala,
st3iny and
tcitworld
as code owners
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #6418 +/- ##
============================================
- Coverage 23.48% 23.47% -0.01%
Complexity 454 454
============================================
Files 247 247
Lines 11705 11708 +3
Branches 2211 2212 +1
============================================
Hits 2749 2749
- Misses 8636 8639 +3
Partials 320 320
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
st3iny
left a comment
st3iny
left a comment
There was a problem hiding this comment.
Breaks the attendees tab on new events: TypeError: Cannot read properties of null (reading 'uri').
st3iny
left a comment
st3iny
left a comment
There was a problem hiding this comment.
Pleas squash and we are good to go.
|
Do we want to backport this (since the issue is from 2020)? Or is this too risky because of the recent shared calendar changes in server? |
Signed-off-by: SebastianKrupinski <[email protected]>
SebastianKrupinski
force-pushed
the
fix/issue-2659-allow-adding-self
branch
from
005b7be to
479ba82
Compare
We can back port this a few versions. |
|
/backport to stable5.0 |
backportbot
bot
added
the
backport-request
backportbot
bot
removed
the
backport-request
3 participants
Resolves: #2659
Modified alreadyInvitedEmails result to NOT exclude the current user by default and to only include actual organize and attendees.