-
Notifications
You must be signed in to change notification settings - Fork 13
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
Fix wrong env var used #224
Conversation
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.
Seems correct. e.g. here.
There's no other mention of AYON_LOCAL_ID
or AYON_SITE_ID
in the docs. So this seems the only place to fix it.
✅
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.
Looks good.
I agree with BigRoy.
@@ -200,7 +200,7 @@ As current implementation relies heavily on Settings and Site Settings, backgrou | |||
|
|||
To do this: | |||
|
|||
- run AYON `Tray` with environment variable AYON_LOCAL_ID set to name of active (source) site. In most use cases it would be studio (for cases of backups of everything published to studio site to different cloud site etc.) | |||
- run AYON `Tray` with environment variable AYON_SITE_ID set to name of active (source) site. In most use cases it would be studio (for cases of backups of everything published to studio site to different cloud site etc.) | |||
- start `Tray` | |||
- check `Local ID` in information dialog after clicking on version number in the Tray |
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.
Quick question: should this be site ID ?
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.
Bonus points if we can add a screenshot showing the id in that dialog and maybe with arrows in the same image pointing to where to click in the tray menu, etc.
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.
No bonus points @kalisp ? :)
There is no
AYON_LOCAL_ID
environment variable, onlyAYON_SITE_ID