Skip to content

Migrate to Angular v19 #414

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

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Conversation

godenji
Copy link

@godenji godenji commented Dec 31, 2024

First step toward standalone component version of the library.

Currently docs have been converted to standalone.

TBD migrating library source itself to standalone. Probably can pull it off by creating base class versions of applicable components and extending with skeleton module/standalone versions for users to import depending on whther or not they've gone all-in on standalone components.

@godenji godenji mentioned this pull request Jan 2, 2025
Copy link
Owner

@michaelbromley michaelbromley left a comment

Choose a reason for hiding this comment

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

Wow thanks so much for taking the time to update all of this! I just spotted a couple of duplicated comments that can be removed.

@@ -261,6 +261,9 @@ describe('PaginationControlsDirective:', () => {
});
});

/**
* Test Component for testing the default controls component
*/
Copy link
Owner

Choose a reason for hiding this comment

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

Duplicated?

@@ -305,6 +309,9 @@ export class DirectiveTestComponent {
}
}

/**
* Test Component for testing bounds correction logic
*/
Copy link
Owner

Choose a reason for hiding this comment

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

Duplicated

Copy link
Author

@godenji godenji Jan 3, 2025

Choose a reason for hiding this comment

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

Done, unfortunately in 2 commits (didn't realize there were two sets of duplicated comments).

@juliengloria
Copy link

Thx guys, any chance to be published soon ? :)

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.

3 participants