Env Simplification, URL-decoding symbols in Procedures#92
Open
masonville17 wants to merge 102 commits into
Open
Env Simplification, URL-decoding symbols in Procedures#92masonville17 wants to merge 102 commits into
masonville17 wants to merge 102 commits into
Conversation
Update ITGlue-Hudu-Migration.ps1
…/false and leave the type conversion in init for interactive questions
…m-may13-26 Chore/pull upstream may13 26
… could be somewhat confusing
…le (which is clever but not kind to changes)
…bug folder (live data means fewer issues_
…viron-opts Chore/normalize environ opts
…ty password in case of vaulting - i'vwe had to manually work around this the last several migrations and I think it better to allow for user to more easily take this path.
…expect. usually these start with that prefix or are blank in such case
…e/pull-upstream-may19
…m-may19 Chore/pull upstream may19
…makes sense here. folks arent going to want to think about something that most likely doesnt apply to them besides
…rocedure-name-simplifyinit Chore/urlencoded procedure name simplifyinit
Contributor
Author
|
I also left this formatting for the imports on supporting tasks or post-tasks
this just makes sure that it can import dependent methods even if post-job is invoked after the fact (otherwise using relative $PSSCriptRoot will fail to resolve dependency path) |
…idance could be helpful, even if this feature is not often used
…ords for and where to place them
Revert "Fix Importing checklists"
…plates, company process templates, company processes, and process runs. I think the expectations for what each of these are need to be better laid out in the README
…yout-collision-task Chore/preflight layout collision task
…ibly) apply it to richtext, embed, heading asset fields, though that might be unnecessary
…l-replace Feat/attachment url replace
…uld be appropriate to convert the process templates to run via kick-off, but I'm starting to think that makes the most sense, so we can preserve due dates and user-assignments.
…ists that have zero-tasks
…ons of having been started in ITGlue
…ns-improve-readme Templates/check runs improve readme
…or asset layout icons
…ection this allows for users to use a color picker to pick color selection f…
setting strictmode explicitly to false in main script scope (and chil…
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Hey- so this is pretty short but let me know if you'd prefer me to break it up at all. It's just a few things that have changed on our end this week -
1- making sure the GUI and env stuff stays decoupled from init
2- simplifying init with existing reusable user-input methods
3- url decoding task and procedure names (we found some ampersands were url-encoded recently)
4- allowing empty passwords option (for users with vaulted / encrypted org passwords, these can show up as blank)
notes on init and envs -
I've set up the environment variables to null-coalesce to the select-objectfromlist method that we already have there. This is just to tidy things up a bit as things grow.
the env vars now just stay as bool
the number -> read-host -> bool assignment in switch works but better to keep things just as bool if thats how they are gonna be used. Plus it makes it so we dont have to make another GUI build if a one changes place with a two in the questionairre
updated gui exe is included which templates envs with these boolean values directly, where applicable