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

Notes / Questions #1

Open
ndushay opened this issue Jan 12, 2024 · 2 comments
Open

Notes / Questions #1

ndushay opened this issue Jan 12, 2024 · 2 comments

Comments

@ndushay
Copy link
Contributor

ndushay commented Jan 12, 2024

  • shouldn't some of the settings live with the gem?
  • there's no language builder?
  • marc_country.rb feels like it belongs in a config folder but whatever
  • workflow migrated to top level workflow_solr_doc and workflow_field?
  • is there an orcid_builder in cocina-models? (how many places does it exist? Peter might be helpful).
  • (ditto for some of the other builders, maybe)

README would be improved with "how to use this" example. Once you figure that out.

@ndushay
Copy link
Contributor Author

ndushay commented Jan 12, 2024

tagging @justinlittman

@justinlittman
Copy link
Contributor

shouldn't some of the settings live with the gem?

This gem only does the building, not the indexing. The only thing that requires configuration is the workflow client, but it is easier to just inject it.

there's no language builder?

It isn't used anywhere.

workflow migrated to top level workflow_solr_doc and workflow_field?

Migrating from a rails structure (with services and models) is imperfect. I don't feel strongly where these go.

is there an orcid_builder in cocina-models?

Maybe, but I'm not trying to refactor here.

README would be improved with "how to use this" example

There is a usage section.

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

No branches or pull requests

2 participants