-
Notifications
You must be signed in to change notification settings - Fork 26
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Error on ng serve due to missing files #38
Comments
We're having trouble reproducing this. When you run the app without that change, are you seeing fields that should be displaying those images but are not? Or, what is the indication you are seeing that those images are missing? |
I've tried to reproduce the issue again, but I haven't managed either... I can't remember the details. I remember that Angular was throwing an error, so I couldn't use the app. The styles.css content was displayed as part of the error both in the browser and in the terminal where I was launching the process, and in the Network tab of the Google Chrome's DevTools I could see that, when trying to download those files from Angular's dev server, it returned a 404. That's why I acknowledged the existance of those files and was able to download them from https://clinicaltables.nlm.nih.gov/lforms-versions/30.0.0-beta.2/webcomponent. Sorry for the inconvenience. By the way, I expected this repo to hold the app deployed here: https://lhcformbuilder.nlm.nih.gov/, but I'm getting this one instead, which is missing some funcionalities and its appearence is different. Am I missing something? |
That is a screenshot of the new "beta" version we are hoping to release soon. I would be interested to hear more about what missing functionality is something you need. If you want the version that is currently live, try version 6.5.2. |
Thanks for the pointing out that we missed Observation-based population. We will try to get that added before we go live with the new version. Thanks also for the feedback about the setting an item to be a group. We were trying to avoid having two ways to do that, but I agree that it the drag-drop method is not as obvious. The initial motivation for this rewrite was to move off of AngularJS, which is no longer being supported, and we took advantage of the rewrite to revise the UI. The whole process has taken much longer than anticipated, and we are trying to get the new version live with at least the same functionality as existed in the old one, before turning to adding new features. We are planning to keep the old version around for a while after releasing the new version, and to provide a link to get to it, but eventually I think there will be some important security issue we can't fix. |
I was having trouble on deploying the app because down_arrow_gray_10_10.png and magnifying_glass.png files where missing. Adding these two elements to filePathMap object on copy-lforms.js solved the issue:
The text was updated successfully, but these errors were encountered: