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.

Sign up for GitHub

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

Jump to bottom

PB-30b: implements toggling active on workspace users #31

Merged
merged 7 commits into from
Sep 14, 2023
Merged

PB-30b: implements toggling active on workspace users #31

merged 7 commits into from
Sep 14, 2023

Conversation

stianjsu
Copy link

@stianjsu stianjsu commented Sep 8, 2023 •
edited
Loading

Implements toggling of active properties on slack users

image

image

@stianjsu stianjsu force-pushed the mutation-reqs-cleanup branch from a848172 to ca527fd Compare September 8, 2023 15:09
@stianjsu stianjsu marked this pull request as draft September 8, 2023 15:31
@stianjsu stianjsu changed the title refactor mutations and implements toggling active PB-30b: implements toggling active on workspace users Sep 8, 2023
@stianjsu stianjsu changed the base branch from mutation-reqs to main September 8, 2023 17:29
@stianjsu stianjsu force-pushed the mutation-reqs-cleanup branch from ca527fd to fac3cc2 Compare September 9, 2023 20:49
@stianjsu stianjsu marked this pull request as ready for review September 9, 2023 20:49
@stianjsu stianjsu force-pushed the mutation-reqs-cleanup branch from fac3cc2 to 270da83 Compare September 9, 2023 21:40
annabjorgo
annabjorgo suggested changes Sep 11, 2023
View reviewed changes
Copy link

@annabjorgo annabjorgo left a comment

Choose a reason for hiding this comment

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

🔥🔥
QualityGIF

sergiosja
sergiosja approved these changes Sep 12, 2023
View reviewed changes
Copy link
Collaborator

@sergiosja sergiosja left a comment

Choose a reason for hiding this comment

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

Veldig pent!

@stianjsu stianjsu force-pushed the mutation-reqs-cleanup branch from 270da83 to a423130 Compare September 13, 2023 13:12
@stianjsu stianjsu requested a review from annabjorgo September 13, 2023 13:29
@stianjsu stianjsu merged commit 2c9eba6 into main Sep 14, 2023
@stianjsu stianjsu deleted the mutation-reqs-cleanup branch September 14, 2023 09:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sergiosja sergiosja sergiosja approved these changes

@JacobTheisen JacobTheisen Awaiting requested review from JacobTheisen

@annabjorgo annabjorgo Awaiting requested review from annabjorgo

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@stianjsu @sergiosja @JacobTheisen @annabjorgo