Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

Allow overriding browser entry points ignore list

See original GitHub issue

Description

Summary: We would like to be able to override the following list: https://github.com/brillout/vite-plugin-ssr/blob/547e5ebaf76b2300457ec85bd027956abba31ef4/vite-plugin-ssr/node/plugin/build.ts#L85

Background:

We are developing an abstraction layer over vite/vite-plugin-ssr. Users can install our npm module, which handles the rendering for certain use cases with minimal code from the user’s side. The entry point to the project and build options are also handled by the npm module. This means in the user’s project, the entry point will be in node_modules, in our module. The line of code linked above is causing us issues as it ignores files in node_modules.

We would like to be able to provide a config option to vite-plugin-ssr to override this ignore list with custom entries.

Implementation:

I wanted to implement this and make a PR but wasn’t sure which approach to take from a design perspective, as the plugin as it is now doesn’t take any options that I could look at for guidance.

Issue Analytics

State:
Created a year ago
Comments:13 (8 by maintainers)

Top GitHub Comments

1reaction

brilloutcommented, Aug 2, 2022

👍 Let me know if you need something. I’m keen on supporting use cases like yours. Note that, as the author of vps, I may find solutions to problems you think are not solvable; so feel free to keep me updated.

0reactions

brilloutcommented, Oct 11, 2022

No offense taken 😃.

I’m looking forward to enabling easy framework building 😃.

On Mon, Jun 13, 2022 at 5:38 PM Hannes Hertach @.***> wrote:

I do not mean to impose a deadline (implied or otherwise) - sorry.

We’re looking forward to the 0.4 release. Until then our patch will be fine. If we end up forking and making changes that we think would benefit general public interest, we would obviously create a PR.

Thank you for your help and for your work!

— Reply to this email directly, view it on GitHub https://github.com/brillout/vite-plugin-ssr/issues/339#issuecomment-1154077909, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAHVQRXNWGK24HJXADWXTUDVO5IWNANCNFSM5YTYPQXA . You are receiving this because you commented.Message ID: @.***>

Top Results From Across the Web

Is there a way to have browsers ignore or override xml ...

Does anyone know a way to disable or override xml-stylesheet directives provided inside an XML document, using Internet Explorer or Firefox? Edit: I've...

Set up Safe Links policies in Microsoft Defender for Office 365

Use URL allow entries in the Tenant Allow/Block List to override the Safe Links URL verdict. In the Manage URLs to not rewrite...

Targets - Parcel

This can be overridden using the context option in the target config (see below). To make Parcel ignore one of these fields, specify...

WebBlocker Override - WatchGuard Technologies

When you enable WebBlocker override in a WebBlocker action, your users can override all denied categories by default. In Fireware v12.5 and higher,...

Multiple Entry Points in Spring Security | Baeldung

Now, let's look at how we can define multiple entry points in our security config. We're going to use an example driven by...