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

Indexes for permissions #248

Merged
merged 8 commits into from
Jun 9, 2023
Merged

Indexes for permissions #248

merged 8 commits into from
Jun 9, 2023

Conversation

fogelito
Copy link
Contributor

No description provided.

@fogelito fogelito requested a review from abnegate March 20, 2023 09:18
@abnegate
Copy link
Member

@fogelito Coming back to this PR, will we need a migration on the Appwrite side for this?

@fogelito
Copy link
Contributor Author

True, looks like the MariaDB
INDEX _permission (_permission,_type,_document)

@abnegate
Copy link
Member

Let's bring this PR up to date and start a corresponding Appwrite PR that includes a migration 👍

…-index

� Conflicts:
�	src/Database/Adapter/Postgres.php
Copy link
Member

@abnegate abnegate left a comment

Choose a reason for hiding this comment

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

Do we have a PR open for the migration on Appwrite yet? Please link it to here and we can merge once both are ready 👍

@fogelito
Copy link
Contributor Author

fogelito commented Jun 4, 2023

No.. Will need some help with this.

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.

None yet

2 participants