-
Notifications
You must be signed in to change notification settings - Fork 7
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
ISSUE #ED-431 feat : Added Program details page #377
base: release-5.2.0
Are you sure you want to change the base?
Conversation
ISSUE #ED-431 feat : Added Program details page
Fetching details from api done in program details page
Added PII consent popup
ISSUE #ED-431 feat : UI changes done for program details page
ISSUE #ED-1560 feat : Files saves for Project evidences
Added toast message for project details page
@@ -9,6 +9,8 @@ import { RouterLinks } from '@app/app/app.constant'; | |||
import { TranslateModule } from '@ngx-translate/core'; | |||
import { FormsModule } from '@angular/forms'; | |||
import { CoreModule } from '../core/core.module'; | |||
import { ProgramDetailsComponent } from './program-details/program-details.component'; | |||
import { CommonConsumptionModule } from '@project-sunbird/common-consumption'; |
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.
why this is required? and what purpose you imported here
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.
As we are using sb-library-filters in program-details page, we need to import the CommonConsumptionModule
}) | ||
export class PiiConsentPopupComponent implements OnInit { | ||
|
||
@Input() msg1 |
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.
give proper and generic names
} | ||
|
||
|
||
converDataSrcToObject(ele) { |
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.
spelling correction
Naming changes in consent popup
Program details page when users have joined and not joined the program