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

chore: adding versioned directory(v0 and v1) #319

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

aryanndwi123
Copy link
Contributor

@aryanndwi123 aryanndwi123 commented Jun 17, 2024

Fixes #320

Describe the changes you have made in this PR -

Versioned Directory Addition

  • Added separate versioned directories (v0 and v1) with their respective src folders.

Screenshots of the changes (If any) -

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

Copy link

netlify bot commented Jun 17, 2024

Deploy Preview for circuitverse ready!

Name Link
🔨 Latest commit 66b27f3
🔍 Latest deploy log https://app.netlify.com/sites/circuitverse/deploys/66786e8a82acd4000934c03c
😎 Deploy Preview https://deploy-preview-319--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.

Copy link

@github-advanced-security github-advanced-security bot left a comment

Choose a reason for hiding this comment

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

ESLint found more than 20 potential problems in the proposed changes. Check the Files changed tab for more details.

// const { activeCircuit } = toRefs(simulatorStore)
name = name || 'Untitled'
name = stripTags(name)
scopeList[id].name = name

Check warning

Code scanning / CodeQL

Prototype-polluting assignment Medium

This assignment may alter Object.prototype if a malicious '__proto__' string is injected from
user controlled input
.
scope.backups[scope.backups.length - 1] !== backup
) {
scope.backups.push(backup)
scope.history = []

Check warning

Code scanning / CodeQL

Prototype-polluting assignment Medium

This assignment may alter Object.prototype if a malicious '__proto__' string is injected from
user controlled input
.
) {
scope.backups.push(backup)
scope.history = []
scope.timeStamp = new Date().getTime()

Check warning

Code scanning / CodeQL

Prototype-polluting assignment Medium

This assignment may alter Object.prototype if a malicious '__proto__' string is injected from
user controlled input
.
*/
function loadModule(data, scope) {
// Create circuit element
var obj = new modules[rectifyObjectType(data.objectType)](

Check failure

Code scanning / CodeQL

Unvalidated dynamic method call High

Invocation of method with
user-controlled
name may dispatch to unexpected target and cause an exception.
@aryanndwi123
Copy link
Contributor Author

aryanndwi123 commented Jun 17, 2024

cc @Prerna-0202 @gr455 @pulkit-30

Copy link
Member

@gr455 gr455 left a comment

Choose a reason for hiding this comment

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

Looks good! Do you think we should name it v1 v2 instead of v0 v1?

@aryanndwi123
Copy link
Contributor Author

Looks good! Do you think we should name it v1 v2 instead of v0 v1?

Since the Vue simulator isn't released yet, it might be better to use v0 as the pre-release version.

@gr455
Copy link
Member

gr455 commented Jun 19, 2024 via email

Copy link
Member

@Prerna-0202 Prerna-0202 left a comment

Choose a reason for hiding this comment

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

LGTM👍

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.

Creating versioned Directories for Vue simulator
3 participants