Skip to content
This repository has been archived by the owner on Dec 7, 2023. It is now read-only.

Group membership #47

Open
wants to merge 17 commits into
base: master
Choose a base branch
from

Conversation

TechandEco
Copy link
Contributor

Creating a new solution which includes a way for users to add users to a Group to give access to resources from a spreadsheet.

@TechandEco
Copy link
Contributor Author

Howdy, this is a new solution ready for review @erickoledadevrel and @asrivas

@erickoledadevrel
Copy link

Assigning @asrivas, since I don't have the bandwidth right now.

@asrivas asrivas requested review from sqrrrl and removed request for asrivas October 9, 2019 17:41
@TechandEco TechandEco changed the title Group membership [DO NOT MERGE YET] Group membership Oct 22, 2019
@TechandEco TechandEco changed the title [DO NOT MERGE YET] Group membership Group membership Jan 1, 2020
@TechandEco
Copy link
Contributor Author

Howdy @asrivas , can I trouble you with reviewing my last solution for the year? Thank you!

group-membership/README.md Outdated Show resolved Hide resolved
group-membership/README.md Outdated Show resolved Hide resolved
group-membership/README.md Outdated Show resolved Hide resolved
group-membership/README.md Outdated Show resolved Hide resolved
group-membership/README.md Show resolved Hide resolved
group-membership/README.md Outdated Show resolved Hide resolved
group-membership/src/Code.js Outdated Show resolved Hide resolved
group-membership/src/appsscript.json Outdated Show resolved Hide resolved
group-membership/src/Code.js Outdated Show resolved Hide resolved
@TechandEco
Copy link
Contributor Author

Thank @asrivas for reviewing, just completed the changes.

group-membership/README.md Outdated Show resolved Hide resolved
group-membership/src/Code.js Outdated Show resolved Hide resolved
group-membership/README.md Outdated Show resolved Hide resolved
group-membership/README.md Outdated Show resolved Hide resolved
group-membership/README.md Outdated Show resolved Hide resolved
group-membership/src/Code.js Outdated Show resolved Hide resolved
group-membership/src/Code.js Show resolved Hide resolved
@TechandEco
Copy link
Contributor Author

Howdy @asrivas I made the final changes, and I think it's ready to ship now. Thanks for your input.

group-membership/README.md Outdated Show resolved Hide resolved
group-membership/src/Code.js Outdated Show resolved Hide resolved
group-membership/src/Code.js Show resolved Hide resolved
group-membership/src/Code.js Show resolved Hide resolved
group-membership/src/Code.js Show resolved Hide resolved
group-membership/src/Code.js Show resolved Hide resolved
group-membership/src/Code.js Show resolved Hide resolved
@TechandEco
Copy link
Contributor Author

Howdy, @asrivas implemented your last requests

group-membership/README.md Show resolved Hide resolved
group-membership/src/Code.js Show resolved Hide resolved
group-membership/src/Code.js Outdated Show resolved Hide resolved
group-membership/src/Code.js Show resolved Hide resolved
@TechandEco TechandEco closed this Mar 27, 2020
@TechandEco TechandEco reopened this Mar 28, 2020
@TechandEco
Copy link
Contributor Author

@asrivas Hoping these changes are ready for the gallery.

@TechandEco TechandEco requested a review from asrivas April 2, 2020 00:26
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants