TagzApp
TagzApp copied to clipboard
FritzAndFriends
For contributors, make a DevContainers configuration
@Spaceshot and I are working on this issue as part of our live stream over on The Dev Talk Show.
Just an update on this. @SpaceShot and I were able to get a working version of TagzApp running in devcontainers in VS Code. We hit this milestone on our show this morning. We still have a few more tasks before I will submit a pull request.
- I need to rebase against the existing code base as Jeff has continued to advance the project.
- I need a README that provides a walkthrough for those who want to develop this way.
- I need an independent test, from @SpaceShot, pulling down the repo and proving 'it works' on his machine.
- I need to make sure the TagzApp.WebTests project still returns green results.
That's all I can think of for now. With any luck we can get PR onto this repo end of next wek. Comment for anything I might have missed.
Don't focus on the WebTests issue yet.. I'm pushing that out for a refactoring with the new Blazor version of the site.
I will be merging the Blazor version and removing the Web project in the next day as I finish the keyboard navigation feature.
Jeff
On Wed, Feb 14, 2024 at 10:26 AM Rich Ross @.***> wrote:
Just an update on this. @SpaceShot https://github.com/SpaceShot and I were able to get a working version of TagzApp running in devcontainers in VS Code. We hit this milestone on our show this morning. We still have a few more tasks before I will submit a pull request.
- I need to rebase against the existing code base as Jeff has continued to advance the project.
- I need a README that provides a walkthrough for those who want to develop this way.
- I need an independent test, from @SpaceShot https://github.com/SpaceShot, pulling down the repo and proving 'it works' on his machine.
- I need to make sure the TagzApp.WebTests project still returns green results.
That's all I can think of for now. With any luck we can get PR onto this repo end of next wek. Comment for anything I might have missed.
— Reply to this email directly, view it on GitHub https://github.com/FritzAndFriends/TagzApp/issues/336#issuecomment-1944060509 or unsubscribe https://github.com/notifications/unsubscribe-auth/AAATF4L662YAQQ3TUOCPKUDYTTJTNBFKMF2HI4TJMJ2XIZLTS2BKK5TBNR2WLJDUOJ2WLJDOMFWWLO3UNBZGKYLEL5YGC4TUNFRWS4DBNZ2F6YLDORUXM2LUPGBKK5TBNR2WLJDUOJ2WLJDOMFWWLLTXMF2GG2C7MFRXI2LWNF2HTAVFOZQWY5LFUVUXG43VMWSG4YLNMWVXI2DSMVQWIX3UPFYGLAVFOZQWY5LFVI2TOMRTGU2TGNJXGOSG4YLNMWUWQYLTL5WGCYTFNSBKK5TBNR2WLKRVG4ZDGNJVGM2TQMNENZQW2ZNJNBQXGX3MMFRGK3ECUV3GC3DVMWVDKOJQGQ2TENRRG44KI3TBNVS2S2DBONPWYYLCMVWKY43VMJVGKY3UL52HS4DFVREXG43VMVBW63LNMVXHJJTUN5YGSY3TSWBKI5DZOBS2U4TFOBXXG2LUN5ZHTJLWMFWHKZNJGY3DKMJSGQ4DGMUCUR2HS4DFUVUXG43VMWSXMYLMOVS2UMRQGE2DKOJYGAZTJAVEOR4XAZNFNRQWEZLMUV3GC3DVMWVDKNZSGM2TKMZVG4ZYFJDUPFYGLJLMMFRGK3FFOZQWY5LFVI2TOMRTGU2TGNJYGGBKI5DZOBS2K3DBMJSWZJLWMFWHKZNKGU4TANBVGI3DCNZYU52HE2LHM5SXFJTDOJSWC5DF . You are receiving this email because you authored the thread.
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub .
Checking in on this issue.. I'll set up a local branch in the repository, and can you PR to that branch? We'll work on the merge and getting the DevContainers working.
Use the feature_devcontainer branch for this PR
I think really the easier path is to add devcontainer support to the project as it stands now. i just wanted it to be a true "ready to go" experience. at the time i think we didnt know how to set up the postgres containers. however, at this point, that may not be the standard. i recall you were working on that day-to-day dev experience and we just need this to fire up what is needed to work on the app today.
