accordproject / models

Accord Project Model Repository
https://models.accordproject.org
Apache License 2.0
29 stars 42 forks source link

Import prefix is missing #153

Open dselman opened 2 years ago

dselman commented 2 years ago

Bug Report 🐛

We've lost the URL prefix for the from imports. I think there's an env var you have to set for that when you generate.

Expected Behavior

Imports should use fully-qualified URLs:

import com.docusign.connect.EnvelopeStatusCode from https://models.accordproject.org/docusign/connect.cto

Current Behavior

import com.docusign.connect.EnvelopeStatusCode from /docusign/connect.cto

Possible Solution

Set the environment variable for the host name and regenerate.

Steps to Reproduce

  1. Browse a model at https://models.accordproject.org
dipit-s commented 2 years ago

@dselman I would like to work on this issue. I have understood what is going wrong here.I will try to find why the variable is not accessible and fix that. (It seems that it has not been assigned in the build.js file) Could you please assign it to me.

apoorvsxna commented 8 months ago

Hi @dselman. Just made a pull request. I think the issue should be resolved now. Let me know if any changes are required.