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

fixed the binding of flags to show create modals #505

Closed
wants to merge 1 commit into from
Closed

fixed the binding of flags to show create modals #505

wants to merge 1 commit into from

Conversation

safwanyp
Copy link
Contributor

This branch was created just for the page that shows databases, but the same issue showed in the collections page.

The PR fixes an issue where the “create new database/collection” in the grid view doesn’t actually open the modal to perform the respective action.

The fix was relative simple. The bindings seemed to be done incorrectly, and all I did was fix these bindings.

What does this PR do?

This PR closes appwrite/appwrite#5992

Test Plan

Manual testing along with scripts.

Related PRs and Issues

appwrite/appwrite#5992

Have you read the Contributing Guidelines on issues?

Yes ✅

This branch was created just for the page that shows databases, but the same issue showed in the collections page.

The PR fixes an issue where the “create new database/collection” in the grid view doesn’t actually open the modal to perform the respective action.

The fix was relative simple. The bindings seemed to be done incorrectly, and all I did was fix these bindings.
@vercel
Copy link

vercel bot commented Aug 15, 2023

@safwanyp is attempting to deploy a commit to the appwrite Team on Vercel.

A member of the Team first needs to authorize it.

@vercel
Copy link

vercel bot commented Aug 20, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
console ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 20, 2023 11:43am

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.

🐛 Bug Report: [Console] Create New Database grid item doesn't work.
2 participants