Skip to content
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

#51 Refactor score logic outside the Vue app #52

Conversation

gscottwilson
Copy link
Contributor

Hello, I work with Chris Turner and our team would like to use your calculator scoring logic for V4 scoring.
In the PR you'll see I've moved the scoring logic outside of the Vue app methods section in order to make it more easily available for 3rd party use (when a full web application is not required). This works for us.

I've attempted to leave the structure of the Vue app the same with no build steps introduced and minimal refactoring.

I've done limited testing comparing several different vector selections to the production V4 calculator and am seeing matching scores and no error messages.

Copy link
Collaborator

@skontar skontar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we should also change line 172 in index.html to force loading new app.js version and not the cached one on some browsers.

@gscottwilson
Copy link
Contributor Author

I think we should also change line 172 in index.html to force loading new app.js version and not the cached one on some browsers.

Good catch! done

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants