Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # 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 👍

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.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants