bcgov / bcrs-shared-components

BCRS Shared Components is a multi-package Lerna repository of shared Vue components, each published individually and that you can explore/ develop/ document/ test using Storybook.
https://bcgov.github.io/bcrs-shared-components/
Apache License 2.0
3 stars 30 forks source link

18752 base-address upgrade #214

Closed PaulGarewal closed 10 months ago

PaulGarewal commented 10 months ago

Issue #: /bcgov/entity#18752

Description of changes:

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the business-filings-ui license (Apache 2.0).

PaulGarewal commented 10 months ago

Not sure if this is what we are looking for (but it is a start):

image

severinbeauvais commented 10 months ago

You will know you have succeeded when this component can be used externally and works properly there. Hopefully it can be used in the same way as the old component. You may need to create a very simple Vue3 UI to test this.

ketaki-deodhar commented 10 months ago

Not sure if this is what we are looking for (but it is a start):

image

This is a good start. Street address and street additional fields should be displayed as well.

severinbeauvais commented 10 months ago

Not sure if this is what we are looking for (but it is a start):

So there's no labels on the fields? Ask UI if we can use it like this (which is presumably the same as in the Assets/PPR UI).

PaulGarewal commented 10 months ago

Not sure if this is what we are looking for (but it is a start):

So there's no labels on the fields? Ask UI if we can use it like this (which is presumably the same as in the Assets/PPR UI).

Ok will do. I will ask Yui.

Here is what the fields look like before entering any values (for reference):

image

PaulGarewal commented 10 months ago

Not sure if this is what we are looking for (but it is a start):

So there's no labels on the fields? Ask UI if we can use it like this (which is presumably the same as in the Assets/PPR UI).

Ok will do. I will ask Yui.

Here is what the fields look like before entering any values (for reference):

image

Confirmed with Yui that we want the same setup as the Assets Team for the BaseAddress.

severinbeauvais commented 10 months ago

Confirmed with Yui that we want the same setup as the Assets Team for the BaseAddress.

Nice! I hoped this would be the case but now it's official.

severinbeauvais commented 10 months ago

You will know you have succeeded when this component can be used externally and works properly there. Hopefully it can be used in the same way as the old component. You may need to create a very simple Vue3 UI to test this.

Any change this will work in a Vue2 UI?

Can you do an address lookup in Storybook? (I know it needs an API key, and we won't want to put that key in any commit in GH as our repos are public!)

Are the props and events the same as the old address component?

cameron-eyds commented 10 months ago

@severinbeauvais @PaulGarewal If your keen on testing in a Vue3 instance before committing, feel free to pull our feature branch: https://github.com/bcgov/ppr/tree/Vue3-Upgrade-Feature-Branch

It's pretty stable, for the most part :) Hit me up for an .env if you want to run it. OR if you want to commit and give it a go as a true import, the above holds true as well 👍

JazzarKarim commented 10 months ago

Due to Paul's computer issues, last commit is from my laptop.