search

FDA-ARGOS / argosdb

BSD 2-Clause "Simplified" License
2 stars 1 forks source link

Push the new ARGOS DB webpage code to production. #20

Closed Reeya123 closed 1 year ago

Reeya123 commented 1 year ago

https://argosdb.org/ -This link shows the old website. New changes have been made and pushed to the main branch. Kindly Update the link

Reeya123 commented 1 year ago

@rajamazumder

rykahsay commented 1 year ago

@Reeya123 @kee007ney @rajamazumder --- I have updated the code for the repo. Anyone should be able to EDIT contents of the pages and deploy changes in any of the argosdb servers (dev/test/beta/prd) following the instructions given in the README.md at the root of the repo.

Reeya123 commented 1 year ago

Hello @rykahsay @HadleyKing . I am unable to follow the steps correctly in order to push my updated code to production. After making changes in the code, I ran the command python3 start.container.py as instructed in the readme.

Error: PS C:\Users\gupta\OneDrive\Desktop\argosdb-main\app> python3 start_app_container.py The syntax of the command is incorrect.

app@0.1.0 build react-scripts build

'react-scripts' is not recognized as an internal or external command, operable program or batch file.

I am assuming this is because it thinks that react script isn't in the node modules but I see it in the package.json file. I don't understand where I am going wrong.

Also after running the above part, exactly where would I make the changes that are asked in the below statement? -->To serve the APP at a given public domain name such as www.argosdb.org, add the following lines to your apache VirtualHost directive (assuming the "app_port" you have selected in the conf/config.json configuration to be 5090)

HadleyKing commented 1 year ago

@Reeya123 I have reassigned this issue to @kee007ney. The instructions in the README.md need to be followed on the ARGOS server. I do not believe you have access.

steph-sing commented 1 year ago

@HadleyKing do you have permission to do this? I don't think that I do @kee007ney are you able to push this today?

kee007ney commented 1 year ago

@steph-sing Robel asked Dacian to give me permissions on the server to do it. Just sent a follow up now to ask for an ETA. Whenever I get permissions I'll push.

steph-sing commented 1 year ago

Thank you for the update Jonathon

kee007ney commented 1 year ago

@Reeya123 New version was pushed. Header portion is removed, but spacing still looks wrong. I'm guessing this is not what you want to see. @HadleyKing will look at the React framework to see if it's an issue here.

kee007ney commented 1 year ago

Hadley, Pat, and I talked this over. Hadley figured out that the HTML was duplicated in /home/keeneyjg/argosdb/app/src/components/home_page.jsx. When we remove that, it works fine. @Reeya123, I don't know if you added that content to that page or not.

Closing this as complete.

steph-sing commented 1 year ago

just checked and looks to be fixed. Thanks guys!