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.

Sign up for GitHub

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

Jump to bottom

Use digitize related hooks from react-util #3507

Closed
wants to merge 35 commits into from
Closed

Use digitize related hooks from react-util #3507

wants to merge 35 commits into from

Conversation

simonseyock
Copy link
Member

Description

Use hooks from terrestris/react-util#472

Related issues or pull requests

Pull request type

  • Bugfix
  • Feature
  • Dependency updates
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Documentation content changes
  • Other (please describe)

Do you introduce a breaking change?

  • Yes
  • No

Checklist

  • I understand and agree that the changes in this PR will be licensed under the
    BSD-2-Clause.
  • I have followed the guidelines for contributing.
  • The proposed change fits to the content of the Code of Conduct.
  • I have added or updated tests and documentation, and the test suite passes (run npm test locally).
  • I have added a screenshot/screencast to illustrate the visual output of my update.

dnlkoch and others added 30 commits September 4, 2023 11:24
@dnlkoch
fix: remove the DigitizeButton
aa8c514 
BREAKING CHANGE: Removes the deprecated DigitizeButton in favour of the DrawButton
@dnlkoch
Merge pull request #3493 from terrestris/remove-digitize-button
7c5e80d 
Remove the deprecated `DigitizeButton` in favour of the `DrawButton`
@annarieger
refactor: remove deprecated HOC components
e66738e 
BREAKING CHANGE: mappify, loadify, isVisibleComponent HOCs as well as MapProvider class were removed
@annarieger
Merge pull request #3496 from terrestris/remove-hocs
e8eeaa8 
Remove some deprecated HOCs
@dnlkoch
chore: update several dependencies
daa2de6
@dnlkoch
chore: 💄
e1ef868
@dnlkoch
chore: set updated plugin name
75a86da
@dnlkoch
build: update enzyme adapter
b9ad1ce
@dnlkoch
build: include TextDecode and TextEncoder
01f9599
@dnlkoch
build: wrap several tests in act()
f65318d
@dnlkoch
build: disable assertions, toBeVisible() isn't working for the model …
76d5d02 
…of antd5
@dnlkoch
build: temporarily disable/remove inkmap dependencies
301c69d
@dnlkoch
build: ignore some context typings (should be fixed as well)
8cc903e
@dnlkoch
fix: make use of dayjs
b0f873c
@dnlkoch
build: antd.variable.css doesn’t exist anymore
89c8749
@dnlkoch
fix: don't pass icon prop
e242cf0
@dnlkoch
ci: run with node 18
c2723b4
@dnlkoch
Merge pull request #3499 from terrestris/dependency-updates
30524c2 
Dependency updates
@annarieger
refactor: remove Panel, Window and Titlebar components
0a602c6 
BREAKING CHANGE: Panel, Window and Titlebar components are not existing anymore
@annarieger
Merge pull request #3497 from terrestris/remove-window
27caa13 
Remove Window, Panel and Titlebar component
@annarieger
refactor: remove toolbar component
7a162a6
@annarieger
Merge pull request #3503 from terrestris/remove-toolbar
b6b3a2f 
refactor: remove toolbar component
@annarieger
refactor: remove user chip component
632cc92
@annarieger
Merge pull request #3504 from terrestris/remove-userchip
11e5ce0 
refactor: remove userchip component
@ahennr
chore: homogenize / sort imports using simple-sort-plugin
a264f71
@ahennr
chore: ignore intelliJ specific files
6aec328
@ahennr
chore: introduce .npmrc
dafd330
@ahennr
Merge pull request #3505 from terrestris/homozenize-imports
d585255 
Eslint: homogenize / sort imports using simple-sort-plugin
@hwbllmnn
refactor: factor out some code to react-util and use it (#3498)
c0066c7
@simonseyock
feat: refactor code into useDraw hook
55798b3
@dnlkoch dnlkoch mentioned this pull request Feb 20, 2024
Merged
15 tasks
@dnlkoch
Copy link
Member

dnlkoch commented Feb 20, 2024

Superseeded by #3749.

@dnlkoch dnlkoch closed this Feb 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
rg-future
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@simonseyock @dnlkoch @annarieger @ahennr @hwbllmnn