bety icon indicating copy to clipboard operation
bety copied to clipboard

added sites_cultivars.rb

Open dlebauer opened this issue 8 years ago • 2 comments

I would like to have sites_cultivars associations be avaialable via api when associations_mode =full_info. I am taking a first pass at enabling this

dlebauer avatar Jan 29 '18 18:01 dlebauer

I would like to have sites_cultivars associations be avaialable via api when associations_mode =full_info. I am taking a first pass at enabling this

On Tue, Mar 20, 2018 at 11:20 AM Scott Rohde [email protected] wrote:

@gsrohde commented on this pull request.

With some minimal changes that I checked in, this will at least allow getting sites-cultivars information in XML, JSON, or CSV format. But note that this uses the old API.

The "new" action does not do anything other than display a dummy SitesCultivar instance (and then only if you use the second displayed request form "GET /cultivars/new.xml" since no default format is defined). The create action doesn't work.

Most likely, you modeled this controller after the CitationsSitesController, which itself doesn't do what it pretends to.

If you let me know what functionality you actually want here, perhaps I can modify this to be useful in the ways that you want it to be.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/PecanProject/bety/pull/569#pullrequestreview-105426677, or mute the thread https://github.com/notifications/unsubscribe-auth/AErs3Bs3m26F_yRro1AW-LQDwUY5DWZqks5tgSxigaJpZM4RxH6b .

betydb avatar Mar 28 '18 00:03 betydb

If I recall, the minimum functionality that we needed here was to be able to access the sites_cultivars relationships. Currently that is difficult to do. This would be nice to have if it is ready to deploy but at this point is low priority.

dlebauer avatar Apr 28 '20 21:04 dlebauer