Skip to content

Conversation

@nickvergessen
Copy link
Member

Steps

  1. Create 70 users:
i=10
while [ $i -ne 70 ]
do
        i=$(($i+1))
        echo "$i"

	USERID=$(uuidgen)
	OC_PASS="123456" occ -vvv user:add --password-from-env --display-name "Dummy #$i" -g "dummies" $USERID
	occ -vvv user:setting $USERID settings email "test$i@yourdomain.tld"

done
  1. Create a conversation as admin with group "dummies" (and check the database queries of this request)

Left is before, right is with this patch
Bildschirmfoto von 2022-02-28 12-09-44

Copy link
Member

@danxuliu danxuliu left a comment

Choose a reason for hiding this comment

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

Code looks good based on the use of userId in resolveRoomDisplayName() (although I have not tested it).

@nickvergessen nickvergessen merged commit 1c633f2 into master Mar 1, 2022
@nickvergessen nickvergessen deleted the techdebt/noid/improve-activity-generation branch March 1, 2022 20:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review feature: api 🛠️ OCS API for conversations, chats and participants performance 🚀 technical debt

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants