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

Index artifacts with no associated projects #1013

Closed
wants to merge 6 commits into from

Conversation

jyoo980
Copy link
Collaborator

@jyoo980 jyoo980 commented Apr 26, 2022

This is part of the general work for #869.

Specifically, this changeset enables the storage of artifacts that have no associated projects.

Helps resolve #869

jyoo980 added 2 commits April 20, 2022 14:47
  * Making the `repository` column in the `artifact` schema NULLABLE.
  * This provides schema-level support for Option[Project.Repository]` fields
    in `Artifact` types.
  * Should help resolve #869.
@jyoo980 jyoo980 self-assigned this Apr 26, 2022
  * Work to enable insertion of artifacts wo. an associated project underway.
  * Still needs work, specifically with the def'n of what a "new" project is.
    * Do we treat a project that later has an associated project "new" now?
    * Details important in implementing the insertArtifact function.
@jyoo980
Copy link
Collaborator Author

jyoo980 commented Apr 26, 2022

Does not compile yet, just pushing up incremental work to get some eyes on it before I might waste time going down a bad path.

jyoo980 added 3 commits April 26, 2022 09:08
  * Will need to create new tests to make sure insertion works with artifacts
    that do not have an associated project.
@jyoo980
Copy link
Collaborator Author

jyoo980 commented May 20, 2022

This is pretty stale, closing and recreating a branch

@jyoo980 jyoo980 closed this May 20, 2022
@jyoo980 jyoo980 deleted the yoo/index-artifacts-no-projects branch May 20, 2022 16:02
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.

Schedule job to update claimed artifacts
2 participants