-
Notifications
You must be signed in to change notification settings - Fork 30
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
🐛 Bug Report: Deploy collection does not maintain the order of the attributes defined in the Appwrite.json file #79
Comments
Hi, thanks for opening this issue. I'm curious, why do you need the order of attributes to be retained? This should not make any difference in practice. |
Sure but sometimes we want (rather than need) to have things clean to read, grouping attribs by feature/meaning, having most important attribs at the start and so on. It's not because we don't need it that we don't want it :-) |
Hmmm we could try to change how we create the collections and attributes:
If we create all collections first, we shouldn't run into any problems with relationship attributes. Theoretically, the create attribute API call should return quickly because the actual creation is asynchronous. |
I sent a pull request with this change. It works perfect for me :) |
As you said, it should make no difference performance wise or anything like that. |
On the Appwrite version 1.6, the user will be able to change the order of attributes defined in the Appwrite.json file. The steps to reproduce will be as follows:
You will be able to confirm the attributes order in your collection are the same as the ones in your appwrite.json file |
👟 Reproduction steps
run
appwrite deploy collection
command👍 Expected behavior
The order of the defined attributes of the appwrite.json file must be maintained when deployed
👎 Actual Behavior
does not maintain the order of the attributes defined in the appwrite.json file they are sorted randomly
🎲 Appwrite version
Version 1.3.x
💻 Operating system
MacOS
🧱 Your Environment
No response
👀 Have you spent some time to check if this issue has been raised before?
🏢 Have you read the Code of Conduct?
The text was updated successfully, but these errors were encountered: