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

Upgrade to React 18 and Improve Types #2298

Merged
merged 10 commits into from
Jun 23, 2022
Merged

Upgrade to React 18 and Improve Types #2298

merged 10 commits into from
Jun 23, 2022

Conversation

MKaHead
Copy link
Contributor

@MKaHead MKaHead commented Jun 22, 2022

Merge Checklist:

  • Code selfreview has been performed (Best Practices).
  • Tests are sufficient.
  • Documentation is up to date.
  • Changelog is up to date.
  • Typescript typings for properties are up to date.
  • Designers approved changes or no approval needed.
  • Dependencies to other components still work.

@MKaHead MKaHead requested a review from amir-ag June 22, 2022 15:25
@MKaHead MKaHead self-assigned this Jun 22, 2022
@MKaHead MKaHead requested a review from selinagahlinger June 23, 2022 09:54
@@ -13,6 +13,7 @@ const selectGermany = async () => {
}, tag);
};

/* TODO #2299
Copy link
Contributor

Choose a reason for hiding this comment

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

Muss hier noch was gemacht werden oder ist der Kommentar/Code gar nicht mehr nötig?

@@ -94,6 +96,7 @@ describe('Input Phone', () => {
await assertChangeValue(`+43 ${validPhoneNumber}`);
});

/* TODO #2299
Copy link
Contributor

Choose a reason for hiding this comment

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

Gleich wie oben

Copy link
Contributor

@selinagahlinger selinagahlinger left a comment

Choose a reason for hiding this comment

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

Denke es passt (hab noch zwei Kommentare verfasst)
@amir-ag / @markus-walther sollten aber sicher nochmals darüber schauen (auch wenn es schon mal gemergt ist 😁 )

@MKaHead MKaHead merged commit 5071146 into develop Jun 23, 2022
@MKaHead MKaHead deleted the feature/improve-types branch June 23, 2022 10:59
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.

2 participants