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

Allow user to remove a foreign key constraint from the column header menu #951

Closed
seancolsen opened this issue Jan 5, 2022 · 2 comments · Fixed by #1354
Closed

Allow user to remove a foreign key constraint from the column header menu #951

seancolsen opened this issue Jan 5, 2022 · 2 comments · Fixed by #1354
Assignees
Labels
ready Ready for implementation type: enhancement New feature or request work: frontend Related to frontend code in the mathesar_ui directory

Comments

@seancolsen
Copy link
Contributor

seancolsen commented Jan 5, 2022

Current behavior

  • To remove a FK constraint, the user must open the Table Constraints dialog.

Desired behavior

  • Find a column associated with a single-column foreign key constraint.
  • Click on the column header to open the column header menu.
  • Choose "Remove Table Link".
  • Observe the same loading/success/error behavior as when clicking on the "Allow NULL" option in the column header menu.

Limitations

  • To delete a multi-column foreign key, you'll still need to do it from the Table Constraints dialog.

Additional context

@seancolsen seancolsen added needs: unblocking Blocked by other work type: enhancement New feature or request work: frontend Related to frontend code in the mathesar_ui directory labels Jan 5, 2022
@seancolsen seancolsen added this to the [08] Working with Views milestone Jan 5, 2022
@seancolsen
Copy link
Contributor Author

@kgodey I put this in the same milestone as all the other FK tickets, but I think we could safely defer it until later. I'll leave it to you to re-prioritize if you so choose.

@kgodey
Copy link
Contributor

kgodey commented Jan 10, 2022

@kgodey I put this in the same milestone as all the other FK tickets, but I think we could safely defer it until later. I'll leave it to you to re-prioritize if you so choose.

I think it's important to do this with the rest of the feature to make it friendlier for non-technical users. It seems like it would be pretty quick to implement.

@kgodey kgodey modified the milestones: [09] Working with Views, [08] Links Between Tables Jan 18, 2022
@seancolsen seancolsen self-assigned this Apr 20, 2022
@seancolsen seancolsen added ready Ready for implementation and removed needs: unblocking Blocked by other work labels Apr 20, 2022
@kgodey kgodey removed this from the [08] Links between Tables milestone Jun 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready Ready for implementation type: enhancement New feature or request work: frontend Related to frontend code in the mathesar_ui directory
Projects
No open projects
Development

Successfully merging a pull request may close this issue.

2 participants