Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
Ralf Brennscheidt
@RalfBrennscheidt
no code changes at all. Wizard only selected MessageExtension
Wictor Wilén
@wictorwilen
Try to run npm install again
Ralf Brennscheidt
@RalfBrennscheidt
nope, same error :(
Wictor Wilén
@wictorwilen
What happens if you do install file-loder?
File-loader
Ralf Brennscheidt
@RalfBrennscheidt
can you see this? -> https://www.paste.org/99119
Strangely it says that the dependency of webpack ^4.... is missing even though i have installed it globally.
Wictor Wilén
@wictorwilen
Have u tried removing node_modules and package-lock.json and npm install?
Ralf Brennscheidt
@RalfBrennscheidt
executed now, still fails with same error.
Ralf Brennscheidt
@RalfBrennscheidt
okay, tried now on my private computer and it worked. Therefore not an issue with the generator. Sorry to bother!
Wictor Wilén
@wictorwilen
npm acts a bit like that once in a while, it misses a package or two during the install
Ralf Brennscheidt
@RalfBrennscheidt
Thank you for your help, Sir!
Wictor Wilén
@wictorwilen
Preview mode again...2.10.0-preview now supports choosing between a configurable and static tab, and also adding more tabs running the generator again
Wictor Wilén
@wictorwilen
10k npm downloads!! Thanks all for your support!
Wictor Wilén
@wictorwilen
And, version 2.10.0 is out
Ralf Brennscheidt
@RalfBrennscheidt
Hi! Is there any news related to SSO ? So there is no need for the whole MS-Graph ping pong-/user-interaction to authenticate ?
Wictor Wilén
@wictorwilen

Hi! Is there any news related to SSO ? So there is no need for the whole MS-Graph ping pong-/user-interaction to authenticate ?

Nothing official AFAIK

Wictor Wilén
@wictorwilen
I just published 2.11.0-preview. It contains support for localization of your apps. You can either start a new project using the 2.11.0 version or you can update a project from 2.9.0+. The latter one I need help with testing on your existing projects, as it will do an update to gulp.config.js - which is a new feature in the generator that allows us to update the core build files going forward, when new features are introduced. Test it on your existing projects - but ensure to have them under source control for any issues in this preview.
Amit
@amitvig22
This is great, and i was able to build a quick teams app and test locally. can someone point me to the documentation to deploy this react app to Azure via visual studio code ? thanks much appreciated.
Wictor Wilén
@wictorwilen
Great to hear @amitvig22. If you take a look at the generated README.md file then you will find instructions for how to deploy it to Azure
Amit
@amitvig22
@wictorwilen aah gotcha, i kept looking at the readme.md for the teams generator and got puzzled. Saw the readme in the project yo teams created and found the instructions.
Andrew Connell
@andrewconnell
Is there a reason why all dependencies, even those not needed in production, are in dependencies section of package.json? IOW, any reason why devDependencies is avoided?
Wictor Wilén
@wictorwilen

Is there a reason why all dependencies, even those not needed in production, are in dependencies section of package.json? IOW, any reason why devDependencies is avoided?

Very valid, that actually has an answer. The reason we did move everything to dependencies is the fact that when you do a git push to Azure and want to avoid having the compiled dist included in Git, meaning Azure should do the build after the push. When Azure does the npm install it did not pick up the devDependencies so the build failed. I have not gone back and tested this for a long time and it was a know issue back then (if I recall with Kudu).

Wictor Wilén
@wictorwilen
So @andrewconnell I have not bothered with this for years, just leaved it that way, since it worked...
But, I just looked into deploy.cmd now and it uses npm install --production (which causes only dependencies to be installed). If I remember I fiddled with that but did not get it to work, might be worth going back and test again #goodfirstissue :-)
Andrew Connell
@andrewconnell
When you run npm install, it looks for an envvar that indicates if you are in PRODUCTION or not (which you can override by including the --production flag too. If not (like your laptop), npm install picks up everything in dependencies & devDependencies. But in An Azure WebApp has the envvar set to PRODUCTION so npm install only grabs what's in dependencies.
In general, you never ship dev stuff (@types/*, gulp, webpack, compilers, test runners, etc)... should only ship what's needed in prod.
First time i'm looking at the teams generator (because i'm building training materials on it) and was a bit surprised, hence why i asked :)
But if you're trying to avoid /dist or an equiv, what you currently have is your only option.
Azure web app's ability to support GIT based deployments are the "easy" button way. Thanks for explaining the thought process behind this.
Andrew Connell
@andrewconnell

You say #goodfirstissue... would you be open to a PR that did two things: (1) moved what's needed => devDependencies & (2) modified the deploy.cmd & docs to explain how to do a deployment that includes both if you use Azure web app capability of git deployment?

This would allow others who want a more enterprisey style deploy to use CI/CD to do the build NOT in production, but only deploy production assets (ie: /dist) which would also keep /dist out of the repo.

That ^^ way you get best of both worlds: you retain the easy deployment option but you don't impose it on people who want to keep dev dep's out of production deployments.
Wictor Wilén
@wictorwilen
Fo sho, I'd love that. And also it's good for you to learn something :-)
Andrew Connell
@andrewconnell
Cool... i'll look into it. Bit swamped writing the training materials now on deadline but something I'll try to get to...
Andrew Connell
@andrewconnell
Is it a known issue that the generator hangs after asking for the default tab name if there are spaces in the tab name?
I don't see it on the issue list, I can bug it if not...
Wictor Wilén
@wictorwilen
Not seen that before, I typically always have a space in there...have u checked OfficeDev/generator-teams#80
Andrew Connell
@andrewconnell
I did... I didn't think it was the same one, but I'll post it as an issue if it keeps up...
Wictor Wilén
@wictorwilen
what node version do you have, I've seen issues with Yeoman/Inquirer related to how async/await is implemented in node 10+
Andrew Connell
@andrewconnell
Node v10, MacOS & only happens on that one question when there's a space in the name.
Wictor Wilén
@wictorwilen
interesting, don't have a mac so can't try. It would be interesting to try without the validation
Andrew Connell
@andrewconnell
I’ll do a proper issue... repro & the like
John Sudds
@jsuddsjr
What are the plans were for OfficeDev/generator-teams in the new year?
Cagdas Davulcu
@cagdasdavulcu
Hello everyone, good question! Is there any feature road map? I checked the product backlog. We have 3 new backlog items and 4 backlog items in progress. Are the backlog items up-to-date or obsolete? It would be great to know which features are planned for 2020, so that i can take one of them and implement.
Wictor Wilén
@wictorwilen

@jsuddsjr , @cagdasdavulcu - I have not made out any detailed plans yet, a lot of things depends on what features Microsoft is releasing and when (they do have some nice stuff coming that will make sense to add)

What's on my list currently is

  • Ensure that stardust/fluent is properly implemented
  • Move to Bot Builder Fx 4.6
  • Deprecate 1.3 and 1.4
  • Improve build pipeline (it is still to slow rebuilding)
  • Update documentation/Wiki
  • Finish the --import feature, that creates a project from a Teams app package created from the App Studio
  • Create the option to import an SPFx project as a Teams Tab
  • Add support for Tab SSO
  • Add support for the tab-loading spinner
  • Also I know @andrewconnell has quite a few opinions on the build tooling/pipeline

Would you be interested in setting up a Teams call on this after the holidays?

Ralf Brennscheidt
@RalfBrennscheidt
Hi Wictor, was nice meeting you at ESPC! I would be very interested in a call.
Cagdas Davulcu
@cagdasdavulcu
Hello everyone, thank you for your response! it is a good idea to setup a Teams call.
Andrew Connell
@andrewconnell

@wictorwilen RE sync in early JAN - LGTM... I'd personally love to tweak the project structure, but not sure if I'm the only one who struggles with the current setup. I'm also relatively new to using it so very well may just be me, but I think it can be simplified & make it more clear what's client vs. what's server.

Also... i've got the startdust stuff implemented with theme switching in tabs. If you like how it's setup in these MS Learn modules I've done, I can take that feature on... https://github.com/OfficeDev/TrainingContent/blob/master/Teams/30%20Create%20Embedded%20Web%20Experiences%20with%20Tabs%20for%20Microsoft%20Teams/Demos/01-learn-msteams-tabs/src/app/scripts/learnPersonalTab/LearnPersonalTab.tsx

Paul Schaeflein
@pschaeflein
I'd be happy to participate in a call in Jan. I've plans for a dotnet template (dotnet new teamsapp) , and I should probably align with the generator. ;)
Cagdas Davulcu
@cagdasdavulcu
Hello Paul, i‘m curious to hear the news about the dotnet core template. Did you already create the git repo?