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

ASD TO DOs #236

Open
5 tasks done
LApetre opened this issue Mar 28, 2023 · 3 comments
Open
5 tasks done

ASD TO DOs #236

LApetre opened this issue Mar 28, 2023 · 3 comments
Assignees

Comments

@LApetre
Copy link

LApetre commented Mar 28, 2023

  • Add the “download template” button also to the main page after the user logs in. This would help getting the template quicker.
  • Add AdviceComponent field to Advice view for cases where there are 2/more advice views per sag entry
  • Save Jwt token on Advice view and pass to download controller
  • If the url does not start with asd redirect to asd on page
  • Change the name of the template to CatchScenarioTable
@LApetre LApetre self-assigned this Mar 28, 2023
@LApetre
Copy link
Author

LApetre commented Mar 28, 2023

Change text on ASD pages:

  1. Changes needed on Advice list site
    AdviceList - Advice View (ices.dk)
  • Replace:

“Here you can find the advice values, and catch options table from ICES fish stock assessments as presented in published advice. You can:”
With:
“Here you can find the advice values and catch scenarios tables from ICES fish stock assessments as presented in the published advice.”

  • Replace:

“In this page you can see all your submissions to the database:”
with:
“Filter advice by:”

  • Change column header “Advice Released Date” to “Publication Date”
  1. Changes needed on the Manage page
    index - Advice View (ices.dk)
  • 1. Replace:

“Here you can find the advice values, and catch options table from ICES fish stock assessments as presented in published advice. You can:”
With:
“Here you can upload and edit the advice information and catch scenarios tables from ICES fish stock assessments.”

  • 2. Replace:

“Summaries available for the Assessment Year of:”
With:
“Advice records available for the Assessment Year:”

  • 3. Replace:

“List of Stock assessment already with Advice View Records:”
With:
“List of stock assessments with advice records:”

  • 4. Replace:

“List of Stock assessment that don't have any Advice View Records:”
With:
“List of stock assessments without advice records:”

@Milldaz
Copy link
Collaborator

Milldaz commented Apr 5, 2023

  • For 'NoAdv' stocks, other fields should not be 'required'

  • Add common name to 'Species' select list: Common name (scientific name)
    -https://sid.ices.dk/Manage/ManageStock.aspx?id=3245 ("Species Scientific Name" and "Species Common Name")

  • Only allow a entry to be published if there is an Advice Release Date, and the current date is later than this (after 12h00 on the day of release).

  • Change "catch option table" to "catch scenario table", see https://asd.ices.dk/viewAdvice/1093 as an example

@colinpmillar
Copy link
Member

colinpmillar commented Apr 25, 2023

Issues encountered developing the R interface

requests:

  • REQUEST: can there be a webservice to download the xml file?
    There is:
    GET request at link https://asd.ices.dk/API/DownloadCatchScenariosXML/{advicekey}
    Need to provide jwt token if the advice record is not published

  • REQUEST: if a token is provided can we access catch options etc of unpublished records?

Otherwise all very cool :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants