-
Notifications
You must be signed in to change notification settings - Fork 9
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
Add "copy from seed realm" checkbox #2077
Conversation
join(this.seedPath!.replace(/\/$/, ''), 'index.json'), | ||
join(realmPath!.replace(/\/$/, ''), 'index.json'), | ||
); | ||
} | ||
} | ||
|
||
let realm = new Realm( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
you are going to have to also make sure to not set the copiedFromRealm
value in the Realm
constructor options below (after you merge from upstream) when copyFromSeedRealm
is false.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Minor nit, but otherwise looks good
Co-authored-by: Hassan Abdel-Rahman <[email protected]>
In this PR, I added a checkbox to the create workspace form to specify whether to copy files from the seed realm.