-
Notifications
You must be signed in to change notification settings - Fork 1
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
Change set_dir and login functions to read DB credientals from enivorment variables. #28
Comments
The way we are going to do this is all changes are going to be made via a pull requests on the |
Sorry Novica, I accidentally deleted the branch. I thought it was something else and didn't check. How much did you loose and do you have a copy of it? |
No worries Petter. All good.
El mié, 24 ene 2024 a las 12:23, Petter Hopp ***@***.***>)
escribió:
… Sorry Novica, I accidentally deleted the branch. I thought it was
something else and didn't check. How much did you loose and do you have a
copy of it?
—
Reply to this email directly, view it on GitHub
<#28 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AABIOXC3RED632SU2QYYEVLYQDVKXAVCNFSM6AAAAAA4TDY4Z2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMBXHEZTIMJTHA>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
--
≠
|
Thanks. I will make a new directory with nearly the same name. |
There is going to be a change in the login to PJS this weekend or next. That may be a nice time to change this. As far as I can see we only need to change a few things
Get the information into .Renvirion, that should be mostly done so that people don't have to type themselves.
Change NVIdb::set_dir_NVI
Change NVIdb::login (group of functions, but they are partly wrappers around one or two core functions. Probably one should create a "subfunction" to retrieve the data in .Renviron.
The text was updated successfully, but these errors were encountered: