Skip to content

Conversation

@imkven
Copy link
Contributor

@imkven imkven commented Oct 3, 2017

@imkven imkven changed the title Assessment submission API [ISDK-75] [ISDK-75] Assessment submission API Oct 3, 2017
imkven added 5 commits October 4, 2017 14:31
# Conflicts:
#	src/pages/achievements/view/achievements-view.page.ts
#	src/pages/activities-classic/list/activities-classic-list.page.ts
#	src/pages/activities/list/list.page.ts
#	src/pages/activities/list/popup.ts
#	src/pages/assessments/assessments.page.ts
#	src/pages/assessments/group/assessments-group.page.ts
#	src/pages/assessments/popup/items-popup.page.ts
#	src/pages/events/checkin/event-checkin.page.ts
#	src/pages/events/list/list.page.ts
#	src/pages/events/view/events-view.page.ts
#	src/pages/forget-password/forget-password.ts
#	src/pages/gallery/gallery.ts
#	src/pages/levels/list/list.ts
#	src/pages/login/login.ts
#	src/pages/magic-link/magic-link.ts
#	src/pages/rankings/list/rankings.page.ts
#	src/pages/rankings/view/ranking-badges.ts
#	src/pages/rankings/view/ranking-details.page.ts
#	src/pages/registration/register.page.ts
#	src/pages/registration/registration.page.ts
#	src/pages/reset-password/reset-password.ts
#	src/pages/settings/leaderboard/leaderboard-settings.page.ts
#	src/pages/settings/settings.page.ts
#	src/pages/team/team.ts
#	src/pages/term-condition/term-condition.page.ts
# Conflicts:
#	src/app/app.module.ts
#	src/pages/forget-password/forget-password.ts
#	src/pages/login/login.ts
#	src/pages/registration/register.page.ts
#	src/pages/registration/registration.page.ts
#	src/pages/reset-password/reset-password.ts
#	src/pages/settings/settings.page.ts
#	src/pages/term-condition/term-condition.page.ts
# Conflicts:
#	src/pages/activities-classic/list/activities-classic-list.page.ts
#	src/pages/registration/register.page.ts
#	src/pages/reset-password/reset-password.ts
public toastCtrl: ToastController,
public assessmentService: AssessmentService,
public submissionService: SubmissionService
private navCtrl: NavController,

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @imkven, when do we use public and when do we use private? May I have a standard answer for this? Some parts of my code are using private, and some parts are using public, thats why I ask la, ^_^

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please change it to public when you do change next time on this file.

@imkven imkven closed this Nov 16, 2017
@imkven
Copy link
Contributor Author

imkven commented Nov 16, 2017

Since this branch is too old. I create another branch for this ticket. #51

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants