Skip to content
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

refactor(date picker): implement date picker as standalone component #1251

Merged

Conversation

domsteinbach
Copy link
Contributor

resolves DEV-2688

@linear
Copy link

linear bot commented Oct 30, 2023

DEV-2688 Refactor datepicker to standalone component

Refactor the datepicker component in the shared lib (https://github.com/dasch-swiss/dsp-das/blob/ab847bd06f9a5eeb5531a8279e370b3e9a109cb1/libs/vre/shared/app-date-picker/src/lib/app-date-picker/app-date-picker.component.ts) to a standalone component.

  • set standalone: true
  • remove ngModule/app-date-picker.module
  • import all dependencies directly into component (imports: [ … )
  • export component in index.ts
  • check tests
  • change import in the app.module.ts of DSP das

@domsteinbach domsteinbach marked this pull request as ready for review October 30, 2023 16:15
@@ -29,7 +28,6 @@ import { PropertyFormItem } from '../../../data-access/advanced-search-store/adv
MatSelectModule,
FormsModule,
ReactiveFormsModule,
AppDatePickerModule,
Copy link
Collaborator

Choose a reason for hiding this comment

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

Please add the import for the AppDatePickerComponent here

@domsteinbach domsteinbach merged commit e89dba8 into main Oct 31, 2023
13 checks passed
@domsteinbach domsteinbach deleted the feature/dev-2688-refactor-datepicker-to-standalone-component branch October 31, 2023 11:51
irmastnt added a commit that referenced this pull request Oct 31, 2023
…return-true-observables-that-also

* main:
  refactor(date picker): implement date picker as standalone component (#1251)
  fix(app/help): fix links to releases (#1253)
  fix(app/resource): removed developer feature that created annotation bug (#1254)
  feat(expert search): implement default gravsearch as placeholder (#1247)
  feat(project description): order project descriptions by language (#1250)

# Conflicts:
#	apps/dsp-app/src/app/project/description/description.component.ts
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants