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

don't designate collection relationship tables as "system" tables. #2043

Merged
merged 1 commit into from
Sep 22, 2022

Conversation

benanhalt
Copy link
Contributor

So that they can be uploaded to through the work bench.

@grantfitzsimmons
Copy link
Member

grantfitzsimmons commented Sep 20, 2022

@grantfitzsimmons
Copy link
Member

grantfitzsimmons commented Sep 20, 2022

For reference:
CollectionRelationship.leftSide has an unhelpful description of leftSide. A better would be Source Collection.

Same for CollectionRelationship.rightSide, which is referred to as Destination Collection in sp6 ui

A more resonable upload:

image

image

image

Am I uploading this incorrectly?

https://freshfish-colrelationship-tables-not-sys.test.specifysystems.org/specify/workbench-plan/7/

@maxpatiiuk
Copy link
Member

No. CO can not be a base table. See #2090 (comment) for explanation.

Collection Relationship Item should be the base table

@maxpatiiuk
Copy link
Member

That is, unless you are expecting to create new Collection Objects

@grantfitzsimmons
Copy link
Member

That is, unless you are expecting to create new Collection Objects

I want to create a new collection object with a relationship to an existing record in this instance.

@maxpatiiuk
Copy link
Member

@maxpatiiuk maxpatiiuk self-requested a review September 22, 2022 17:37
Copy link
Member

@maxpatiiuk maxpatiiuk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Uploading though WB worked for me

@grantfitzsimmons grantfitzsimmons merged commit d076910 into production Sep 22, 2022
@grantfitzsimmons grantfitzsimmons deleted the colrelationship-tables-not-sys branch September 22, 2022 20:48
@specifysoftware
Copy link

This pull request has been mentioned on Specify Community Forum. There might be relevant details there:

https://discourse.specifysoftware.org/t/specify-7-7-3-release-announcement/774/1

@specifysoftware
Copy link

This pull request has been mentioned on Specify Community Forum. There might be relevant details there:

https://discourse.specifysoftware.org/t/specify-7-7-4-release-announcement/807/1

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.

Ability to bulk associate collection object relationships from the form
4 participants