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

Make org_with_dept_id example match the builtin form #365

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mmeisel
Copy link

@mmeisel mmeisel commented Mar 29, 2023

I was using this example as a reference (as I believe is the intention) when trying to add a single field to organizations without changing anything else. But the actual fields used by ckan (at least in 2.9) are named "description" and "image_url" rather than "notes" and "url". Since these are used by ckan on the organizations page, it's important they remain unchanged. So I think it would be better if the example uses the correct names.

While I was at it, I also made sure everything was using the correct preset (when available), made sure the labels matched, and increased the scheming_version to match the rest of the documentation.

@wardi
Copy link
Contributor

wardi commented Apr 19, 2023

@mmeisel would you merge master so we can get the tests running on this PR?

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.

2 participants