Skip to content

don't encourage saving secrets in source control #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

Closed
wants to merge 1 commit into from

Conversation

MichaelHedges
Copy link

modified:   .gitignore
modified:   README.md
renamed:    eg-03-csharp-auth-code-grant-core/appsettings.json -> eg-03-csharp-auth-code-grant-core/example-appsettings.json

	modified:   .gitignore
	modified:   README.md
	renamed:    eg-03-csharp-auth-code-grant-core/appsettings.json -> eg-03-csharp-auth-code-grant-core/example-appsettings.json
@InbarGazit InbarGazit requested a review from MattKingDS January 3, 2020 22:40
@InbarGazit InbarGazit closed this Apr 16, 2020
@AaronWDS AaronWDS deleted the MH/dont-check-secrets-in branch November 1, 2020 11:17
AaronWDS added a commit that referenced this pull request Dec 31, 2020
* Improving the JWT flow to ensure it works correctly for developers even if not using Quickstart

* removing extra file

* Adding Rooms API to the code examples

* Quickstart fix

* updating scopes to be based on the API selected

* Quickstart adjustment

* Fixing typo

* Quickstart adjustments

* fixing broken links from DEVDOCS-3396

* Fixing manifest for VS Extensions

* Small refactoring to fit steps of code example

* Updating README.md with information about Rooms API

* fix typos in rooms examples

* Fixing links from inside Launcher to the different code example controllers

* Fixing issue 24 (#24)

* Removed "ceremony"

* Updating versions of the SDKs

* removed "sandbox"

* Update PAYMENTS_INSTALLATION.md

* Adding three new Advanced Recipient Routing (ARR) code examples

* Fixing issues

* cleanup

* adding three new ARR (Advanced Recipient Routing) code examples

* more more part needed for ARR

* Fix typo

* Update ConfigurationManifest.json

* Rename EnvelpeDocItem.cs to EnvelopeDocItem.cs

* fixed homepage view, removed 3,4 from controllers and views files and updated readme to remove those examples

Co-authored-by: Inbar Gazit <inbar.gazit@docusign.com>
Co-authored-by: Paige Rossi <Paige.Rossi@dsa026975.docusignhq.com>
Co-authored-by: Paige Rossi <70241979+paigesrossi@users.noreply.github.com>
Co-authored-by: meihDS <70775251+meihDS@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants