finos / FDC3

An open standard for the financial desktop.
https://fdc3.finos.org
Other
201 stars 132 forks source link

Upgrade FDC3 website to docusaurus >=3.5.2 #1371 #1418

Open sivaprasadanakarla opened 2 weeks ago

sivaprasadanakarla commented 2 weeks ago

This update contains all the issue fixes highlighted during review.

Notes from the original PR (#1409):

This PR contains all the files modified for docusaurus migration. steps followed:

  1. delete yarn.lock

  2. rm -rf node_modules package-lock.json

  3. update package.json a) "@docusaurus/core": "~3.5.2", b) "@docusaurus/preset-classic": "~3.5.2", c) "@docusaurus/theme-mermaid": "~3.5.2", d) "react": "^18.2.0", e) "react-dom": "^18.2.0",

  4. npm install. – ( notice- found 0 vulnerabilities)

  5. npm run build. ( now build will fail)

  6. npx docusaurus-mdx-checker (iscans and shows compilation issues of .md files)

  7. Fixed all the .md files and 1 js file which is generating .md file

  8. npm run build (success)

  9. npm run start (application is up)

linux-foundation-easycla[bot] commented 2 weeks ago

CLA Signed

The committers listed above are authorized under a signed CLA.

netlify[bot] commented 2 weeks ago

Deploy Preview for fdc3 canceled.

Name Link
Latest commit 2b58c524392e76ffdc2a4fea28f32dffadb43b15
Latest deploy log https://app.netlify.com/sites/fdc3/deploys/672afdb874cffd0008b22024
sivaprasadanakarla commented 2 weeks ago

Hi @robmoffat @bingenito ,

[this PR is raised as part of citi_hackathon event] Could you please review this latest PR for Issue#https://github.com/finos/FDC3/issues/1371. We have fixed all the issues highlighted in below PR's.

https://github.com/finos/FDC3/pull/1409 https://github.com/finos/FDC3/pull/1406

Kindly review and suggest if any other issues.

Thanks in advance.

Best Regards, Shiva.

kriswest commented 1 week ago

Hi, you may need to rebase this PR on main and re-run the checks as we've just merged a big PR for a new web-based specification. Hopefully we've not introduced too many extra links using the <{link}> syntax, although I know there are some in the references page. There are likely others.

kriswest commented 1 week ago

@sivaprasadanakarla please let us know if you or another participant have some time in the next few days to update this PR from main and make sure several new documents that were recently added (a big PR that had been in flight for many months just merged in) are also all good. We're very grateful that you took this task on and are keen to merge the result and get the website to a clean dependency audit!