Skip to content

Conversation

@devlinjunker
Copy link
Contributor

@devlinjunker devlinjunker commented Sep 30, 2022

PR for Making all Component files use Typescript

Moved out of #1907

(File changes in this PR contains the updates from #1907 until it is merged)

Commits with actual changes:

Steps

  • Move all components use Vue.extend({ ...
  • Add types/ directory with .vue files containing type definitions used throughout app

Up Next

@devlinjunker devlinjunker changed the title Move Component Files to Typescript [Vue Rewrite] Move Component Files to Typescript Sep 30, 2022
@Grotax Grotax added the Skip-Changelog No changelog update is required, minor change label Sep 30, 2022
Copy link
Member

@Grotax Grotax left a comment

Choose a reason for hiding this comment

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

Looks good, after the other PR is merged

Copy link
Member

@Grotax Grotax left a comment

Choose a reason for hiding this comment

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

needs to be rebased :)

@devlinjunker devlinjunker force-pushed the components-to-typescript branch from 2709767 to a9aba8a Compare October 4, 2022 02:47
@devlinjunker
Copy link
Contributor Author

rebased, double check to make sure and then we can merge

@Grotax Grotax merged commit fe62ff2 into nextcloud:vue-rewrite Oct 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Skip-Changelog No changelog update is required, minor change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants