Skip to content

Revive Exports to MyCoPortal#4034

Open
JoeCohen wants to merge 26 commits intomainfrom
3249-revise-mcp-observation-download-formats
Open

Revive Exports to MyCoPortal#4034
JoeCohen wants to merge 26 commits intomainfrom
3249-revise-mcp-observation-download-formats

Conversation

@JoeCohen
Copy link
Copy Markdown
Member

@JoeCohen JoeCohen commented Mar 14, 2026

Updates observation Reports to support imports to MyCoPortal, which now uses Symbiota 3.x.

@JoeCohen JoeCohen linked an issue Mar 14, 2026 that may be closed by this pull request
24 tasks
@JoeCohen JoeCohen self-assigned this Mar 14, 2026
@JoeCohen JoeCohen added the MyCoPortal export MO obss to MyCoPortal label Mar 14, 2026
@coveralls
Copy link
Copy Markdown
Collaborator

coveralls commented Mar 14, 2026

Coverage Status

coverage: 96.105%. remained the same
when pulling b175c26 on 3249-revise-mcp-observation-download-formats
into 9cc6eba on main.

@JoeCohen JoeCohen changed the title Remove verbatim_attributes from MyCoPortal expot format Revive Exports to MyCoPortal Mar 14, 2026
@JoeCohen JoeCohen linked an issue Mar 14, 2026 that may be closed by this pull request
24 tasks
JoeCohen and others added 19 commits March 14, 2026 14:58
Aligns data with correct labeled columns
- Its value duplicates that of catalogNumber.
- The MCP upload UI :
  - Has no option to select `dbpk` as the primary key.
  - Does allow `catalogNumber` to be selected as the primary key
  - Does not present a field to which `dbpk`can be mapped.
- It works fine without `dbpk`
This reverts commit 6a78de6.
Adds a comment
- Fixes test_location_with_observations_regions ordering after DST change
- Applies order_by_default to the observation_query scope in the test so
both sides of assert_query_scope are ordered consistently. MySQL returns
rows in arbitrary order without an ORDER BY, which was exposed by the
spring DST transition.
Including changing formatting for readability, fixing errors
@JoeCohen JoeCohen marked this pull request as ready for review March 19, 2026 15:34
Copilot AI review requested due to automatic review settings March 19, 2026 15:34
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates Mushroom Observer’s MyCoPortal export reports/documentation to align with Symbiota 3.x import expectations, removing fields/behaviors that no longer apply and clarifying the export workflow.

Changes:

  • Remove the verbatimAttributes column (and its linked-URL payload) from the MyCoPortal data CSV export.
  • Update tests and UI strings to reflect the revised MyCoPortal export format names (“MyCoPortal Data”, “MyCoPortal Images”).
  • Refresh the MyCoPortal update README with revised Symbiota 3.x import steps/links.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 7 comments.

File Description
app/classes/report/mycoportal.rb Updates MyCoPortal CSV schema/output to drop verbatimAttributes and refresh Symbiota docs references.
test/classes/report_test.rb Adjusts expectations to match removed verbatimAttributes and updated Associated Taxa docs link.
config/locales/en.txt Renames download option labels to match the new MyCoPortal report naming.
README_MYCOPORTAL_UPDATE.md Updates the operational export/import procedure for Symbiota 3.x and the renamed report formats.

You can also share your feedback on Copilot code review. Take the survey.

JoeCohen and others added 6 commits March 19, 2026 08:40
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Responds to CoPilot suggestions
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

MyCoPortal export MO obss to MyCoPortal

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Revise MCP Observation Download Formats

3 participants