Skip to content
This repository was archived by the owner on Jul 14, 2026. It is now read-only.

Update ODS to 2.1.2 - #4594

Merged
kulmann merged 11 commits into
masterfrom
ods-table-component
Feb 2, 2021
Merged

Update ODS to 2.1.2#4594
kulmann merged 11 commits into
masterfrom
ods-table-component

Conversation

@kulmann

@kulmann kulmann commented Jan 8, 2021

Copy link
Copy Markdown
Contributor

Description

This PR updates ODS to 2.1.2
We needed to do some fixes because of changed things in ODS:

  • oc-table is now oc-table-simple
  • oc-star doesn't exist anymore
  • moment has been removed from ODS. We want to use luxon now instead - replacing first occurrence of moment in this PR (because it was used from ODS, which doesn't provide moment anymore), others will follow in a separate one.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Technical debt
  • Tests

Checklist:

  • Code changes
  • Unit tests added
  • Acceptance tests added
  • Documentation ticket raised:

@update-docs

This comment has been minimized.

@kulmann kulmann self-assigned this Jan 8, 2021
@kulmann
kulmann requested a review from LukasHirt January 8, 2021 11:37
@kulmann
kulmann force-pushed the ods-table-component branch 2 times, most recently from 4b980fb to fdaefc0 Compare January 19, 2021 13:42
@kulmann kulmann changed the title use oc-table-simple instead of oc-table Update ODS to 2.1.0 Jan 19, 2021
Comment thread apps/draw-io/src/DrawIoEditor.vue
@kulmann
kulmann force-pushed the ods-table-component branch 2 times, most recently from 36258ed to 34a2d13 Compare January 27, 2021 12:43
@kulmann
kulmann force-pushed the ods-table-component branch from 4c2fe5a to 12089ef Compare January 29, 2021 17:54
@kulmann

kulmann commented Jan 29, 2021

Copy link
Copy Markdown
Contributor Author

Rebased after #4671 got merged

@kulmann
kulmann marked this pull request as ready for review January 29, 2021 17:55
@kulmann kulmann added the Status:Needs-Review Needs review from a maintainer label Jan 30, 2021
@kulmann

kulmann commented Jan 30, 2021

Copy link
Copy Markdown
Contributor Author

Finally green CI. Please re-review @LukasHirt 😃

@kulmann kulmann changed the title Update ODS to 2.1.0 Update ODS to 2.1.2 Feb 2, 2021

@LukasHirt LukasHirt left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM 🚀

@kulmann
kulmann merged commit fbdeaa5 into master Feb 2, 2021
@delete-merged-branch
delete-merged-branch Bot deleted the ods-table-component branch February 2, 2021 10:25
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Status:Needs-Review Needs review from a maintainer

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants