Skip to content

Conversation

@drgrice1
Copy link
Member

Fix the remaining incorrect Ref usages.

Update the remaining import statements to use the provided quasar webpack alias.

Fix my mistake with the jest configuration. I seem to have mixed the two methods I suggested, instead of only using one. I installed require-json5, but didn't use it. I did intend to use that, rather than adding all of the aliases in jest.config.js, and so that is done here.

@drgrice1
Copy link
Member Author

Wait a bit with this.

@drgrice1
Copy link
Member Author

Okay, this is ready.

Update the remaining import statements to use the provided quasar
webpack alias.

Fix my mistake with the jest configuration.  I seem to have mixed the
two methods I suggested, instead of only using one.  I installed
require-json5, but didn't use it.  I did intend to use that, rather than
adding all of the aliases in jest.config.js, and so that is done here.
Copy link
Member

@drdrew42 drdrew42 left a comment

Choose a reason for hiding this comment

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

Thanks for this @drgrice1 -- just so I'm clear on the changes: neither boot/ nor component/ require leading src/. Are those the only two?

And are we looking to merge this in now, and resolve conflicts with #63?

@drgrice1
Copy link
Member Author

There are a few others also. All of the aliases provided by quasar are listed at https://quasar.dev/quasar-cli/handling-webpack#webpack-aliases.

Yeah, I think this should be merged, and then the conflicts in #63 resolved.

@drdrew42 drdrew42 merged commit 8b760f7 into openwebwork:main Nov 17, 2021
@drgrice1 drgrice1 deleted the cleanup branch November 17, 2021 16:31
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.

2 participants