glygener / glygen-issues

Repository for public GlyGen tickets
GNU General Public License v3.0
0 stars 0 forks source link

List of other pages for the biomarker portal #1237

Closed ReneRanzinger closed 2 months ago

ReneRanzinger commented 7 months ago

Go over all the pages in GlyGen that are reachable from:

And decide which of these pages are needed for the biomarker portal as well. Classify them into pages that are needed for the first version (home) and pages that can be added in a second version (supersearch).

After you are done assign the ticket to @sujeetvkulkarni. Based on your selection he will compile a list of web services that are needed beyond the biomarker specific services.

sujeetvkulkarni commented 5 months ago

@DaniallMasood please check https://glygen.ccrc.uga.edu/frontend/ for biomarker portal. Please check which pages you want to keep/remove/add etc. It currently uses glygen test apis and would require vpn.

DaniallMasood commented 5 months ago

@sujeetvkulkarni I would like to keep the pages on there (search, data, api, and sparql). These look fine to me thank you

DaniallMasood commented 5 months ago

I am adding a check list of things that need to be removed/added/changed to the front end @sujeetvkulkarni

Will provide content where necessary in another ticket. Please let me know if we can do these and when it can be expected. Also if you have any comments or suggestions on this as well.

DaniallMasood commented 5 months ago

@sujeetvkulkarni Raja and I would like these pages up by Friday. So if possible these changes could be made by Thursday morning?

ReneRanzinger commented 5 months ago

@DaniallMasood , @rajamazumder No, Sujeet is traveling today and we have the all hands meeting the rest of the week.

DaniallMasood commented 5 months ago

@ReneRanzinger @sujeetvkulkarni no problem then, that completely slipped my mind, sorry about that. Monday works for us as well if possible

sujeetvkulkarni commented 5 months ago

@DaniallMasood Like @ReneRanzinger said I was traveling to DC today and have whole day face to face meetings for rest of week. Will try to see what changes will be possible to fix by Monday.

DaniallMasood commented 5 months ago

@sujeetvkulkarni What i meant to say was that you can start this on Monday since you are busy this week sorry for the confusion. Whenever you get to this just let me know if you have questions about anything.

sujeetvkulkarni commented 5 months ago

@DaniallMasood ok sure, will get back to you if have any questions in this regard.

DaniallMasood commented 5 months ago

@sujeetvkulkarni is it possible to have a designation that this is a test website anywhere on the pages? maybe at the top? or have water mark on the pages?

sujeetvkulkarni commented 4 months ago

@DaniallMasood For API and SPARQL cards/links will there be some change or they will remain same.

DaniallMasood commented 4 months ago

@sujeetvkulkarni for now they will remain the same. We do not have SPARQL page yet but when we do we can link it. So if it is removable then it can removed for now.

For API card/link if @seankim658 has completed the swagger page we can link to that

DaniallMasood commented 4 months ago

@sujeetvkulkarni Also the date for the soft launch that was discussed in the biomarker partnership is June 28th, which is this Friday. Do you think this is doable by then?

sujeetvkulkarni commented 4 months ago

@DaniallMasood I am working on it currently and will finish it by then. You want it by Friday end of the day or Friday morning?

DaniallMasood commented 4 months ago

Friday morning if possible but if not then later on Friday is good too. Thank you @sujeetvkulkarni

DaniallMasood commented 4 months ago

@sujeetvkulkarni what is the update on these tickets? thank you

sujeetvkulkarni commented 4 months ago

@DaniallMasood Most of the changes are done. I am going through final evaluations, will check in the changes, initiate the build and let you know once build is ready for testing today.

sujeetvkulkarni commented 4 months ago

@DaniallMasood added grant number to footer as per Rene's suggestion.

sujeetvkulkarni commented 4 months ago

@DaniallMasood Done. GitHub - 4777f226163f8a664b4ef1eb643ec417da9c65f0. https://glygen.ccrc.uga.edu/frontend/ Please check and let me know.

DaniallMasood commented 4 months ago

@sujeetvkulkarni everything seems to be working from my end. Thank you so much fir getting this done.

sujeetvkulkarni commented 4 months ago

@DaniallMasood Great! I have added a TEST WEBSITE tag on the top of website, it will appear on all pages.

DaniallMasood commented 4 months ago

@sujeetvkulkarni thank you so much for all this. Just two things. Could we actually remove the sparql card and option in the drop down menu? Also the user setting that appears at the top still says GlyGen. Can that be changed to Biomarker Portal or removed? image

sujeetvkulkarni commented 4 months ago

@DaniallMasood Removed sparql from header, footer and card. Used Biomarker Partnership portal or Biomarker instead of GlyGen wherever necessary - please check.

DaniallMasood commented 4 months ago

looks good! thank you