hudson-newey commented 2 years ago

Add Support for Harvest Naming

What is the purpose of this PR? This PR adds a name attribute to Harvest models, displays this name in the harvest list for each project, and adds a way to edit the name of the harvest from the detailed harvest view.

Changes

Adds a Harvest name column to the harvest list view
Adds the Harvest name in the harvest title component
Creates a new shallow updateName() Harvest service
Separates isAbortable() canUpdate() Harvest model methods

Problems

No

Issues

Fixes #1996

Visual Changes

Harvest list view with name in far left column

An example harvest detail view with pencil edit button next to the project & harvest name

View after the pencil edit button is clicked

Toast notification once the name is changed

Final Checklist

[x] Assign reviewers if you have permission
[x] Assign labels if you have permission
[x] Link issues related to PR
[x] Ensure project linter is not producing any warnings (npm run lint)
[x] Ensure build is passing on all browsers (npm run test:all)
[x] Ensure CI build is passing
[x] Ensure docker container is passing (docs)

github-actions[bot] commented 2 years ago

Size Change: +1.75 kB (0%)

Total Size: 2.67 MB

Filename	Size	Change
`dist/workbench-client/browser/main.****************.js`	845 kB	+734 B (0%)
`dist/workbench-client/server/main.js`	1.77 MB	+1.01 kB (0%)

ℹ️ View Unchanged

| Filename | Size | | :--- | :---: | | `dist/workbench-client/browser/assets/environment.json` | 555 B | | `dist/workbench-client/browser/index.html` | 3.83 kB | | `dist/workbench-client/browser/manifest.json` | 150 B | | `dist/workbench-client/browser/polyfills.****************.js` | 12.5 kB | | `dist/workbench-client/browser/runtime.****************.js` | 648 B | | `dist/workbench-client/browser/styles.****************.css` | 37.4 kB |

_{compressed-size-action}

github-actions[bot] commented 2 years ago

Unit Test Results

        6 files         6 suites 18m 43s :stopwatch: 17 394 tests 16 830 :heavy_check_mark: 564 :zzz: 0 :x: 17 490 runs 16 926 :heavy_check_mark: 564 :zzz: 0 :x:

Results for commit eddf23e9.

:recycle: This comment has been updated with latest results.

hudson-newey commented 2 years ago

I've updated the PR with the UI changes in Chrome

I'm currently troubleshooting some alignment irregularities which exist when using Firefox and will update the PR code once this is fixed

QutEcoacoustics / workbench-client