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

Auto-completion suggests attributes already present #1326

Closed
Jolg42 opened this issue Dec 12, 2022 · 0 comments · Fixed by #1537
Closed

Auto-completion suggests attributes already present #1326

Jolg42 opened this issue Dec 12, 2022 · 0 comments · Fixed by #1537
Labels
bug/2-confirmed Bug has been reproduced and confirmed. kind/bug A reported bug. team/schema Issue for team Schema. topic: autocompletion
Milestone

Comments

@Jolg42
Copy link
Member

Jolg42 commented Dec 12, 2022

Similarly to #1079

Screenshot 2022-12-12 at 09 13 42

model hello {
  id Bytes @id() @default("") @ignore @unique @map("") @relation() 
}

Note, this combination is definitely not expected to work. But the completion could, in general, remove the already present attributes.

Here we could do the same as we do for @map(), also for @unique, @default(), @ignore, @relation()

@Jolg42 Jolg42 added bug/1-unconfirmed Bug should have enough information for reproduction, but confirmation has not happened yet. kind/bug A reported bug. topic: autocompletion team/schema Issue for team Schema. labels Dec 12, 2022
@Druue Druue added bug/2-confirmed Bug has been reproduced and confirmed. and removed bug/1-unconfirmed Bug should have enough information for reproduction, but confirmation has not happened yet. labels Sep 19, 2023
Druue added a commit that referenced this issue Sep 19, 2023
@Jolg42 Jolg42 added this to the 5.4.0 milestone Sep 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug/2-confirmed Bug has been reproduced and confirmed. kind/bug A reported bug. team/schema Issue for team Schema. topic: autocompletion
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants