-
Notifications
You must be signed in to change notification settings - Fork 65
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
Post for updates in Okta Angular SDK #1337
Conversation
✅ Deploy Preview for okta-blog ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
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.
Short, sweet, and to the point -- beautiful!
382da52
to
021fda1
Compare
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.
Demonstrating editorial changes to show Colton the github review process :)
export class AppModule { } | ||
``` | ||
|
||
This method will continue to work, and you can continue using the module as you have it now without any changes. |
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.
This method will continue to work, and you can continue using the module as you have it now without any changes. | |
This method will continue to work, and you can keep on using the module as you have it now without any changes. |
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.
@edunham maybe a fine line, but I'd like to keep it as continue. Trying to encourage the pattern going forward should change, and keep on sounds more positive than continue to me?
d7327c5
to
5d32728
Compare
This blog post has the following features: