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

Replace flint example #81

Closed
wants to merge 1 commit into from
Closed

Replace flint example #81

wants to merge 1 commit into from

Conversation

jpjpjp
Copy link

@jpjpjp jpjpjp commented Feb 1, 2020

Replace flint example with webex-node-bot-framework sample

Copy link
Collaborator

@ObjectIsAdvantag ObjectIsAdvantag left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks for your contribution,
please check:

  1. there is an issue with the TOC in the PR, on line 23/25
  2. in awesome, we don't typically replace but rather discuss adding a new resource and removing others, so please create separate issues for these 2 discussions

@jpjpjp
Copy link
Author

jpjpjp commented Feb 4, 2020

This Webex Bot Starter is a direct replacement for the old Spark Bot Starter that is still using a brand we changed several years ago. The functionality is essentially one for one, but the new sample uses correct branding and is based on a supported framework (as the original author of flint is no longer supporting it).

The change was discussed with the original author. I leave it to the Awesome moderators to determine if its better to maintain old, poorly branded, no longer supported samples or to periodically refresh them over the years as the product moves.

@jpjpjp
Copy link
Author

jpjpjp commented Feb 19, 2020

thanks for your contribution,
please check:

1. there is an issue with the TOC in the PR, on line 23/25

2. in awesome, we don't typically replace but rather discuss adding a new resource and removing others, so please create separate issues for these 2 discussions

Hi, based on the conversation in this issue: #83, perhaps we can pull the trigger on this?

@jpjpjp
Copy link
Author

jpjpjp commented Jul 3, 2020

Hey @ObjectIsAdvantag, I noticed this morning that this PR never got completed. As I mentioned I've checked in with the original owners of the flint project and the old sparkbot starter. These are no longer maintained and they have blessed the replacements. @nmarus and @valgaze, please feel free to let @ObjectIsAdvantag know if I'm misrepresenting anything here.

@valgaze
Copy link

valgaze commented Jul 3, 2020

@ObjectIsAdvantag @jpjpjp

All sounds good, go for it

@WebexSamples WebexSamples closed this by deleting the head repository May 3, 2024
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.

4 participants