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

Add objective contributors create/remove functions #851

Merged
merged 3 commits into from
Sep 12, 2023

Conversation

simenheg
Copy link
Collaborator

Add create and remove functions for the new objective contributors collection.

@simenheg simenheg requested a review from a team August 31, 2023 12:00
@simenheg simenheg force-pushed the objective-contributors-add-delete branch 4 times, most recently from 8dfddff to 5f7cfc7 Compare September 5, 2023 13:34
Copy link
Collaborator

@petterhj petterhj left a comment

Choose a reason for hiding this comment

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

LGTM!

firestore.rules Outdated
Comment on lines 92 to 95
/**
* Return true if the current user is a member of the parent of the
* document's objective *before* performing the action.
*/
Copy link
Collaborator

Choose a reason for hiding this comment

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

Har du sett noe rundt syntaks for kommentarer i CEL? Er det bare // som egentlig er akseptert? (Min syntax-highlighter er ikke så glad, men er heller ikke så veldig farlig.)

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Ah, hm. Ser ut som de andre kommentarene blir ignorert også, men da bytter jeg dem om.

},
objective: {
type: 'reference',
required: false,
Copy link
Collaborator

Choose a reason for hiding this comment

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

Denne skal ikke være påkrevd?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Jo. ;)

Add create and remove functions for the new objective contributors
collection.
@simenheg simenheg force-pushed the objective-contributors-add-delete branch from 5f7cfc7 to e63e700 Compare September 12, 2023 11:41
@simenheg simenheg force-pushed the objective-contributors-add-delete branch from e63e700 to f801fe4 Compare September 12, 2023 11:50
@simenheg simenheg merged commit f183642 into main Sep 12, 2023
3 checks passed
@simenheg simenheg deleted the objective-contributors-add-delete branch September 12, 2023 12:19
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.

2 participants