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

feat(street-search): replace linkStreet sub-form with street select #702

Merged
merged 3 commits into from
Mar 11, 2024

Conversation

derrabauke
Copy link
Contributor

This will resolve the required connection between locality and street for creating new buildings/entrances.

Added in locations

  • create new building
  • create new entrance

Preview

Disabled state

image

Enabled

image

Search

image

Selected

image

This will resolve the required connection between locality and street
for creating new buildings/entrances.
@derrabauke
Copy link
Contributor Author

@kaldras @luytena ready for review

Copy link
Contributor

@luytena luytena left a comment

Choose a reason for hiding this comment

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

Looking good 👌 Do we still need the link-street route? Otherwise, we could remove it, I think. Also, did you get a chance to test it integrated in one of the ebau cantons?

addon/components/search-street.hbs Show resolved Hide resolved
addon/components/search-street.js Show resolved Hide resolved
addon/components/search-street.js Show resolved Hide resolved
addon/components/search-street.js Show resolved Hide resolved
translations/de.yaml Show resolved Hide resolved
@luytena
Copy link
Contributor

luytena commented Mar 4, 2024

@derrabauke Have you had time to take a look at the review comments?

@derrabauke
Copy link
Contributor Author

at the review comments?

Will have a look today

Copy link
Contributor Author

@derrabauke derrabauke left a comment

Choose a reason for hiding this comment

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

@luytena please re-review

addon/components/search-street.js Show resolved Hide resolved
translations/de.yaml Show resolved Hide resolved
addon/components/search-street.js Show resolved Hide resolved
addon/components/search-street.hbs Show resolved Hide resolved
@derrabauke
Copy link
Contributor Author

Looking good 👌 Do we still need the link-street route? Otherwise, we could remove it, I think. Also, did you get a chance to test it integrated in one of the ebau cantons?

Yes we still need the link-street route for editing existing entrances.

@luytena
Copy link
Contributor

luytena commented Mar 11, 2024

Looking good 👌 Do we still need the link-street route? Otherwise, we could remove it, I think. Also, did you get a chance to test it integrated in one of the ebau cantons?

Yes we still need the link-street route for editing existing entrances.

Mhh, is this because it uses a different request (modifyBuildingEntrance)? From a user perspective I think it would be nice to have the same interface everywhere. But I think we can discuss this if the need arises at a later point.

Copy link
Contributor

@luytena luytena left a comment

Choose a reason for hiding this comment

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

LGTM 🔥

@luytena
Copy link
Contributor

luytena commented Mar 11, 2024

@derrabauke Let's release it and test it integrated in eBau.

@derrabauke derrabauke merged commit 73e1785 into inosca:main Mar 11, 2024
10 checks passed
@derrabauke derrabauke deleted the feat-street-select branch March 11, 2024 17:10
@luytena
Copy link
Contributor

luytena commented Mar 11, 2024

🎉 This PR is included in version 7.5.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants