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.

Sign up for GitHub

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

Jump to bottom

[AzureDeploy] Enable deploy options for newer samples #136

Closed
wants to merge 49 commits into from
Closed

[AzureDeploy] Enable deploy options for newer samples #136

wants to merge 49 commits into from

Conversation

pcostantini
Copy link
Contributor

Changes

  • All Samples - Read configuration settings from Web.Config (C#) or Environment Vars (Node)
  • Add Azure Deploy buttons to newer samples (both sample's README and index README)

NOTE: This PR depends on merging the following PRs first:

ejadib and others added 30 commits May 17, 2017 11:55
@ejadib
[C#-RollerSkill] Add README.md
963f91b
@pcostantini
Remove extra chars from More Information link
3b477fa
# Avoiding checking for default selected option (not all subsets have…
c5cf67b 
… a default)
# Updating README file - adding code highlights and detailed sample flow
8259155
# Checking that no more than one shipping option is selected
2da2779
# Reviewing C# README file
5d3958b
# Fixing typo in C# README file
ac0bdf3
# Adding bot emulator image @ outcome details
62cac5d
# Updating payments flow image
89f847c
# Exporting tokenFormat values at PaymentToken
91e47c8
# Fixing validations on payment confirmed
ba9ac5c
# Uploading README images
4e63786
# Uploading README file for node payments sample
02db5b8
# Removing no required requisite and fixing typo
f2cb6ca
@pcostantini
Merge 'node-sample-payments-readme' into payments-readmes
9f40803
@pcostantini
Fix Lint issues
ce79999
@pcostantini
Load .env with MS AppId/Password
46ebe3c
@pcostantini
Update to BotBuilder v3.8.2
59571f5
@pcostantini
[Node] RollerSkill README updates
460df7e
@pcostantini
Merge 'csharp-rollerskill-readme' into rollerskill-readmes
8ea9395
@pcostantini
Renames
c9c347a
@pcostantini
Link Node samples
4b59ae3
@pcostantini
Fix typo
e36bbe8
@pcostantini
Link C# samples
dfd7117
@pcostantini
Fix lint issues
d43ea92
@pcostantini
Update to support Azure Deploy button
ba3b6e1
@pcostantini
Update to support Azure Deploy button
4bab45f
@pcostantini
Update to support Azure Deploy button
1439c55
@pcostantini
Update to support Azure Deploy button
3b08400
@pcostantini
Update to support Azure Deploy button
4464636
pcostantini and others added 19 commits May 22, 2017 21:24
@pcostantini
Update to support Azure Deploy button
873167a
@pcostantini
Merge remote-tracking branch 'remotes/origin/sample-payments-readmes'…
15083a7 
… into azuredeploy-updates
@pcostantini
Update to support Azure Deploy button
8fec898
@pcostantini
Updates to More Information links
563f6fb
@pcostantini
Merge remote-tracking branch 'remotes/origin/sample-payments-readmes'…
8a8ca9a 
… into azuredeploy-updates
@pcostantini
Update to support Azure Deploy button
8e8239d
@pcostantini
Update to support Azure Deploy button
38d700a
@pcostantini
Update to support Azure Deploy button
9bf9bbf
@pcostantini
Update to support Azure Deploy button
e948000
@pcostantini
Update to support Azure Deploy button
ff8a301
@pcostantini
Update to support Azure Deploy button
6358565
@pcostantini
Update to support Azure Deploy button
62ce826
@pcostantini
Update to support Azure Deploy button
0d346e4
@pcostantini
Merge remote-tracking branch 'origin/rollerskill-readmes' into azured…
af143b3 
…eploy-updates
@pcostantini
Update to support Azure Deploy button
1e24191
@pcostantini
Update to support Azure Deploy button
64f80fd
@pcostantini
Update to support Azure Deploy button
b0bcb21
@pcostantini
Update to support Azure Deploy button
ad1c7c9
@pcostantini
Update to support Azure Deploy button
da6531c
@pcostantini pcostantini deleted the azuredeploy-updates-nodelab branch May 23, 2017 20:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@pcostantini @msftclas @ejadib