-
Notifications
You must be signed in to change notification settings - Fork 59
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
Added example region-tags for app-profile object #288
Added example region-tags for app-profile object #288
Conversation
Codecov Report
@@ Coverage Diff @@
## master #288 +/- ##
=======================================
Coverage ? 100%
=======================================
Files ? 10
Lines ? 1269
Branches ? 0
=======================================
Hits ? 1269
Misses ? 0
Partials ? 0
Continue to review full report at Codecov.
|
f90f300
to
f8fde9c
Compare
0b8ec4f
to
f23c34f
Compare
const appProfile = instance.appProfile(appProfileId); | ||
|
||
appProfile | ||
.create() |
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
494a7d6
to
b6a520b
Compare
@vijay-qlogic, can you please fix the |
f1f5a8f
to
9c07bc6
Compare
@vijay-qlogic @sduskis this PR is getting a little moldy. Can we close this out? Is there more work to be done here? |
1 similar comment
@vijay-qlogic @sduskis this PR is getting a little moldy. Can we close this out? Is there more work to be done here? |
@JustinBeckwith, it looks like @vijay-qlogic addressed my comments. I'm going to approve. |
Fixes #157 partially for app-profile object (it's a good idea to open an issue first for discussion)