Skip to content

Reduce Requested Permission for new comers#42

Closed
jordan-ae wants to merge 2 commits into
ubiquity:developmentfrom
jordan-ae:fix/reduce-requested-permissions
Closed

Reduce Requested Permission for new comers#42
jordan-ae wants to merge 2 commits into
ubiquity:developmentfrom
jordan-ae:fix/reduce-requested-permissions

Conversation

@jordan-ae
Copy link
Copy Markdown
Contributor

Resolves #41

@jordan-ae jordan-ae requested a review from pavlovcik as a code owner April 11, 2024 05:40
@jordan-ae
Copy link
Copy Markdown
Contributor Author

@0x4007 @gentlementlegen This should reduce requested permission for new comers

@gentlementlegen gentlementlegen self-requested a review April 11, 2024 05:51
Comment thread .vscode/extensions.json Outdated
Comment thread .vscode/settings.json Outdated
Comment thread database.types.ts Outdated
Comment thread package-lock.json
Comment thread supabase/config.toml Outdated
return;
}

const { data, error: fetchError } = await supabase.from("users").select("userType").eq("id", user.id).single();
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

We do not have such column for users. However their id matches their GitHub ID. From there, you can fetch the user details and check they level inside the Organization, member or collborator etc.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

The architecture of the project and how all the pieces communicate is a bit confusing to me. Please how can I get the schema correctly. Thanks

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

The project architecture looks something similar to this
ubiquibot/.github#11 (comment)

And the Supabase schema can be found at
https://github.com/ubiquity/ubiquibot/tree/development/supabase

@rndquu
Copy link
Copy Markdown
Member

rndquu commented May 14, 2024

Closing this one, already fixed by #48

@rndquu rndquu closed this May 14, 2024
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.

Lower requested permissions on user registration

3 participants