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.

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

[JS] [Auth] Update F5 flow in ME and Bot SSO sample to provision Auth #783

Closed
3 tasks done
Tracked by #669
singhk97 opened this issue Nov 7, 2023 · 1 comment · Fixed by #928
Closed
3 tasks done
Tracked by #669

[JS] [Auth] Update F5 flow in ME and Bot SSO sample to provision Auth #783

singhk97 opened this issue Nov 7, 2023 · 1 comment · Fixed by #928
Assignees

Comments

@singhk97
Copy link
Collaborator

singhk97 commented Nov 7, 2023

  • Update ME sso sample
  • Update Bot sso sample
  • Update Adaptive Cards sso sample
@singhk97
Copy link
Collaborator Author

#911 the teams toolkit flow for setting up auth can be found in this sample

singhk97 pushed a commit that referenced this issue Nov 30, 2023
## Linked issues

closes: #783 

## Details

Provide a list of your changes here. If you are fixing a bug, please
provide steps to reproduce the bug.

#### Change details

> Describe your changes, with screenshots and code snippets as
appropriate

- add settings to F5/provision/deploy with Teams Toolkit

**code snippets**:

**screenshots**:

## Attestation Checklist

- [x] My code follows the style guidelines of this project

- I have checked for/fixed spelling, linting, and other errors
- I have commented my code for clarity
- I have made corresponding changes to the documentation (we use
[TypeDoc](https://typedoc.org/) to document our code)
- My changes generate no new warnings
- I have added tests that validates my changes, and provides sufficient
test coverage. I have tested with:
  - Local testing
  - E2E testing in Teams
- New and existing unit tests pass locally with my changes

### Additional information

> Feel free to add other relevant information below
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 a pull request may close this issue.

3 participants