Skip to content

Conversation

@JammingBen
Copy link
Contributor

@JammingBen JammingBen self-assigned this Jun 22, 2022
@JammingBen JammingBen force-pushed the batch-invite-action branch from 88c7a35 to 6458fb0 Compare June 23, 2022 12:46
@DeepDiver1975 DeepDiver1975 force-pushed the feat/invite-guests branch 2 times, most recently from c148b82 to b5cbf52 Compare June 23, 2022 14:51
@JammingBen JammingBen force-pushed the batch-invite-action branch from 6458fb0 to d0e327f Compare June 24, 2022 07:12
@DeepDiver1975 DeepDiver1975 force-pushed the feat/invite-guests branch 2 times, most recently from 4d9c6ab to 809d8f0 Compare June 27, 2022 21:04
@JammingBen JammingBen force-pushed the batch-invite-action branch from d0e327f to cdc39cf Compare June 29, 2022 08:55
@jnweiger jnweiger mentioned this pull request Jul 12, 2022
42 tasks
@JammingBen JammingBen marked this pull request as ready for review July 12, 2022 17:32
@JammingBen
Copy link
Contributor Author

This one is ready for review, sorry for not marking earlier. It needs #521 to be finished and merged first though, otherwise we permanently have merge conflicts with each force push to #521

@phil-davis
Copy link
Contributor

I put this into the Classic Server Team Board project, and mentioned some people for review.

1 JS test failed - I have no idea if that is a real fail.

There is a conflict reported, so that n eeds to be sorted out.

continue;
}

// only add new users
Copy link
Member

Choose a reason for hiding this comment

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

it might be better to use a set instead of an array. The operation should be more clear too:

var s = new Set()
s.add(user)  // guarantees uniqueness

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I tried it, unfortunately Sets do not support uniqueness with objects.

var user = users[i].trim();
var userAdded = false;

for (var j = 0; j < foundUsers.length; j++) {
Copy link
Member

Choose a reason for hiding this comment

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

same as above, it seems better to use a set instead of an array

@JammingBen JammingBen force-pushed the batch-invite-action branch from 3c6b2c8 to f4e8ae5 Compare July 18, 2022 08:32
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@JammingBen JammingBen requested a review from jvillafanez July 18, 2022 09:40
@DeepDiver1975 DeepDiver1975 merged commit d3ce514 into feat/invite-guests Jul 18, 2022
@delete-merged-branch delete-merged-branch bot deleted the batch-invite-action branch July 18, 2022 13:21
@jnweiger
Copy link
Contributor

Confirmed implemented in 0.7.0-rc.1
image

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.

6 participants