erddap icon indicating copy to clipboard operation
erddap copied to clipboard

conversion of string builder to jte for htmldataset output page

Open Harish-2003 opened this issue 3 months ago • 5 comments

we are currently rendering the htmldataset output page using string builder as we are planning to migrate the pages to jte i would like to create the jte version of htmldataset output page

Harish-2003 avatar Sep 04 '25 17:09 Harish-2003

@Harish-2003 Just making sure you saw the new jte branch: https://github.com/ERDDAP/erddap/tree/jte

We are intending to do future work in that branch. It is merged with current main as of yesterday as well.

ChrisJohnNOAA avatar Sep 05 '25 12:09 ChrisJohnNOAA

https://github.com/Harish-2003/erddap/blob/882c9419502124d5e3e5b41b2825918c98567f50/WEB-INF/classes/gov/noaa/pfel/erddap/Erddap.java#L7287 ??

Harish-2003 avatar Sep 07 '25 10:09 Harish-2003

https://github.com/Harish-2003/erddap/blob/882c9419502124d5e3e5b41b2825918c98567f50/WEB-INF/classes/gov/noaa/pfel/erddap/Erddap.java#L7287 ??

I'm not sure what the question is here

ChrisJohnNOAA avatar Sep 09 '25 03:09 ChrisJohnNOAA

https://github.com/Harish-2003/erddap/blob/882c9419502124d5e3e5b41b2825918c98567f50/WEB-INF/classes/gov/noaa/pfel/erddap/Erddap.java#L7287 is this the page that we are intending to convert?

Harish-2003 avatar Sep 19 '25 16:09 Harish-2003

I think the html dataset output page is referring to this: https://oceanview.pfeg.noaa.gov/erddap/tabledap/erdCalCOFINOAAhydros.htmlTable?cruise%2Cship%2Cship_code%2Corder_occupied%2Clatitude%2Clongitude%2Cline%2Cstation%2Ctime%2Cctd_depth%2Ctemperature%2Csalinity%2Coxygen%2Cpercent_saturation&latitude%3E=29.84667&latitude%3C=51.09333&longitude%3E=-132.2883&longitude%3C=-117.2033&time%3E=2014-08-16T00%3A00%3A00Z&time%3C=2014-08-23T02%3A40%3A00Z

This is generated from https://github.com/ERDDAP/erddap/blob/main/WEB-INF/classes/gov/noaa/pfel/erddap/filetypes/HtmlTableFiles.java

There's a code path for EDDTable datasets and another for EDDGrid.

ChrisJohnNOAA avatar Sep 20 '25 01:09 ChrisJohnNOAA