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.

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

Group webid #43

Merged
merged 3 commits into from
May 19, 2021
Merged

Group webid #43

merged 3 commits into from
May 19, 2021

Conversation

bourgeoa
Copy link
Contributor

this solves issues #38 #40

@bourgeoa bourgeoa requested a review from timbl April 28, 2021 14:57
@bourgeoa
Copy link
Contributor Author

bourgeoa commented May 4, 2021

@timbl

  • updated to use sameAs (thoroughly tested with rdflib patched with all latest PR's).
  • added
    • webid url check
    • refresh of card groupList (needed when deleting a card webId
    • remove card from index when deleting a card

Remains to define the contact-pane migration strategy.

if (!book) {
book = kb.any(undefined, ns.vcard('includesGroup'))
if (!book) {
throw new Error('findBookFromGroups: Cant find address book which this group is part of')
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
throw new Error('findBookFromGroups: Cant find address book which this group is part of')
throw new Error('findBookFromGroups: Cannot find address book which this group is part of')

@bourgeoa bourgeoa merged commit e5d6416 into main May 19, 2021
@timea-solid timea-solid deleted the groupWebid branch March 11, 2022 09:59
@bourgeoa bourgeoa linked an issue Apr 2, 2022 that may be closed by this pull request
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.

Restore the juggling of webids to that Group ACLs work again
2 participants