search

stats4sd / aec_portfolio

A proof of concept for the AEC Consortium Project Management / Assessment System
GNU General Public License v3.0
0 stars 0 forks source link

Update Excel Template File for Importing Projects #292

Closed dan-tang-ssd closed 4 days ago

dan-tang-ssd commented 5 days ago

This PR is submitted to fix #291

This PR contains below changes:

  1. Add new column "exchange_rate_eur" in excel template file for importing projects
  2. Shift validations to corresponding columns after adding new column at column G

The static excel template file "storage\app\Agroecology Funding Tool - Initiative Import Template.xlsx" is reference only. This is no longer being used when downloading the excel template file for importing projects.

I thought we are using static excel template file in PR #251, while it has been changed to generate excel template file programmatically in PR #247

dan-tang-ssd commented 4 days ago

Hi @dave-mills , I am going to deploy this PR to staging env.

I did a checking in Forge, it looks like there was an issue in your last deployment on 2024-09-11. It looks like something wrong when running command "npm run build". I remote to server and run the same command, got same error.

Are you working on this issue?

Screen shots:

image

image

image

image

dave-mills commented 4 days ago

Thanks Dan,

Looks like I didn't remove that un-used JS file from the vite build list after I deleted the file. I've pushed a fix to the dev branch and it's now successfully deployed.

Cheers, Dave

From: dan-tang-ssd @.> Sent: Wednesday, September 18, 2024 10:49 To: stats4sd/aec_portfolio @.> Cc: Dave Mills @.>; Mention @.> Subject: Re: [stats4sd/aec_portfolio] Update Excel Template File for Importing Projects (PR #292)

Hi @dave-millshttps://github.com/dave-mills , I am going to deploy this PR to staging env.

I did a checking in Forge, it looks like there was an issue in your last deployment on 2024-09-11. It looks like something wrong when running command "npm run build". I remote to server and run the same command, got same error.

Are you working on this issue?

Screen shots:

image.png (view on web)https://github.com/user-attachments/assets/d8934c40-8f74-476a-bc9c-9b883c185dc1

image.png (view on web)https://github.com/user-attachments/assets/4802c10f-41e1-483c-8df4-be9100722018

image.png (view on web)https://github.com/user-attachments/assets/963492ba-0d84-444d-baca-f5506d555df6

image.png (view on web)https://github.com/user-attachments/assets/e8dd1f00-aa5e-4e06-9a05-5910bd27dbac

—

Reply to this email directly, view it on GitHubhttps://github.com/stats4sd/aec_portfolio/pull/292#issuecomment-2358007549, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABLSJ7LKBXJV4Y3REDSFAQTZXFEBZAVCNFSM6AAAAABOLOSF26VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNJYGAYDONJUHE.

You are receiving this because you were mentioned.

dan-tang-ssd commented 4 days ago

Staging env deployment is completed. Performed testing with positive result.

I will submit another PR for live env deployment.