-
Notifications
You must be signed in to change notification settings - Fork 588
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
Move the 'windows app' page into target doc space #8434
Conversation
@@ -0,0 +1,29 @@ | |||
# MakeCode Windows app | |||
|
|||
Some MakeCode editors also have an associated Windows app which contains |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this referring to the Electron app? I think electron isn't just Windows (but this IS for the windows app page....so feel free to ignore 😅) And I'm not sure how we refer to the Electron app externally -- offline app?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OH This is about the windows app! Sorry ignore me
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What's publish method for the Electron app vs. the Windows app? Windows app store?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
electron app I think we just have a link to on our site. lemme grab a link...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
https://makecode.microbit.org/offline-app yeah it's linked to in the offline page
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this app alternative info will go in a pxt-microbit/windows-app.md
extended page after this is merged.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ooo gotcha
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm late in doing this. Should've coordinated before microsoft/pxt-microbit#4184 went in. Better nate than lever.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ah sorry D:
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
* move page to common-docs * update the page info a bit * the 'the'
The page is moved into
./common-docs
fromdocs
in order for it to be extensible to the targetwindows-app.md
pages. The page is updated with extend sections and a bit more text.RE: microsoft/pxt-microbit#4159, microsoft/pxt-microbit#4184