Skip to content

Some fixes to the Angular Tutorial#102

Merged
NathanWalker merged 4 commits intoNativeScript:developfrom
DmitrySharabin:tutorial-angular
Sep 24, 2022
Merged

Some fixes to the Angular Tutorial#102
NathanWalker merged 4 commits intoNativeScript:developfrom
DmitrySharabin:tutorial-angular

Conversation

@DmitrySharabin
Copy link
Contributor

Hey there, 👋🏻

Thank you so much for the awesome work you do by developing NativeScript.

I worked through the tutorial and found a couple of issues. It would be nice if we could fix them. 😊

Thank you!

According to the folder structure stated in the Folder structure section, the assets folder is in the app folder, not in the src one.
Since the app template provides app.css but not app.scss, we need to provide the styles in the correct format.
@cla-bot
Copy link

cla-bot bot commented Sep 24, 2022

Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. In order for us to review and merge your code, please sign the CLA at https://www.nativescript.org/cla.
CLA has not been signed by users: @DmitrySharabin.
After signing the CLA, you can ask me to recheck this PR by posting @cla-bot check as a comment to the PR.

@DmitrySharabin
Copy link
Contributor Author

@cla-bot check

@cla-bot cla-bot bot added the cla: yes label Sep 24, 2022
@cla-bot
Copy link

cla-bot bot commented Sep 24, 2022

The cla-bot has been summoned, and re-checked this pull request!

@NathanWalker
Copy link
Contributor

Thank you @DmitrySharabin ❤️

@NathanWalker NathanWalker merged commit b9fcecc into NativeScript:develop Sep 24, 2022
@DmitrySharabin DmitrySharabin deleted the tutorial-angular branch September 25, 2022 06:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants