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

Javascript import maps #2108

Closed
wants to merge 3 commits into from
Closed

Conversation

niklasmohrin
Copy link
Member

Closes #2031

I haven't touched the typescript files yet, because I haven't decided between

  • trying to change the imports in the typescript files from ./csrf-utils.js to csrf-utils and teaching typescript how to resolve these modules, and
  • just using ./csrf-utils.js in the import map too.

I think I am slightly preferring the latter. What do you think?

@richardebeling
Copy link
Member

richardebeling commented Jan 29, 2024

Both fine with me. Using csrf-utils.js (possibly with leaing ./) also in inline scripts might look a bit more natural, so I'd even slightly prefer that

@niklasmohrin
Copy link
Member Author

Closing this for now, as it does not seem too important and I would rather be around if this breaks something :D

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

Successfully merging this pull request may close these issues.

Use import maps
2 participants