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

Enable the Main simulator to dynamically load the Vue simulator #332

Closed

Conversation

aryanndwi123
Copy link
Contributor

Describe the changes you have made in this PR -

  • Ensure getConfiguredVersion handles query parameter simver for version selection.
  • Introduce loadVersion function for dynamic loading of main.ts based on configured version.
  • Update application entry point to use version-specific main.ts.
  • Improve entry point to enhance version control and flexibility.

Screenshots of the changes (If any) -

Final result :

(In the video, I labeled the navbar as v0 and v1 next to the project name for demonstration purposes, although these versions are not reflected in the actual code):

Screencast.from.2024-07-06.21-35-18.webm

Note: Please check Allow edits from maintainers. if you would like us to assist in the PR.

Copy link

netlify bot commented Jul 6, 2024

Deploy Preview for circuitverse ready!

Name Link
🔨 Latest commit e5074eb
🔍 Latest deploy log https://app.netlify.com/sites/circuitverse/deploys/6696a50c7e0a7e000812fbfa
😎 Deploy Preview https://deploy-preview-332--circuitverse.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

console.log(`Loaded ${version}`);
})
.catch(err => {
console.error(`Failed to load ${version}`, err);

Check failure

Code scanning / CodeQL

Use of externally-controlled format string High

Format string depends on a
user-provided value
.
@JoshVarga
Copy link
Collaborator

Closing since this is combined into another PR

@JoshVarga JoshVarga closed this Jul 28, 2024
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