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

21977 Implemented Pending list #683

Merged
merged 4 commits into from
Jul 17, 2024
Merged

Conversation

severinbeauvais
Copy link
Collaborator

@severinbeauvais severinbeauvais commented Jul 16, 2024

Issue #: bcgov/entity#21977

This is another incremental code change for the subject ticket.

Description of changes:

  • app version = 7.3.14
  • now store pending bootstrap item to Pendings list
  • added "pendings" store state, action, getter
  • added pending conditionals to dashboard v-cards
  • refactored PendingList.vue
  • implemented PendingTemplate.vue
  • implemented ContinuationIn.vue
  • misc cleanup

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).

@severinbeauvais severinbeauvais self-assigned this Jul 16, 2024
@severinbeauvais severinbeauvais marked this pull request as draft July 16, 2024 20:41
@bcgov bcgov deleted a comment from bcregistry-sre Jul 17, 2024
- now store pending bootstrap item to Pendings list
- added "pendings" store state, action, getter
- added pending conditionals to dashboard v-cards
- refactored PendingList.vue
- implemented PendingTemplate.vue
- implemented ContinuationIn.vue
- misc cleanup
this.storeAddresses({ data: data.offices || [] })

// set parties
this.storeParties({ data: { parties: data.parties || [] } })
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Although this bootstrap filing isn't complete, we want to show the addresses and parties (both disabled) on the page, ie:

image

Copy link
Collaborator

@ArwenQin ArwenQin left a comment

Choose a reason for hiding this comment

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

LGTM!

notify you of the results by email. You can also come back here to check on the progress.
</p>
</div>
</template>
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

This filing only needs to override the default (empty) body.

@@ -69,7 +69,7 @@
:class="{'invalid-section ml-n1 pl-1 rounded-0': showInvalidSection(item)}"
>
<div class="list-item">
<div class="todo-label">
<div class="todo-label pt-1">
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

This (and line 2307) is a very subtle change but it better aligns the View Details button and the Continue In button.

Before:

image

After:

image

}
if (this.isContinuationInTodo) return this.getTasks[0]?.task.filing.displayName
if (this.isContinuationInPending) return this.getPendingsList[0].displayName
if (this.isContinuationInFiling) return this.getFilings[0]?.displayName
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

^^ get Display Name from correct list

Example when filing is in Pending list:

image

@@ -49,6 +49,8 @@ export interface RootStateIF {
parties: Array<PartyIF>
recordsAddress: OfficeAddressIF
registeredAddress: OfficeAddressIF
tasks: Array<ApiTaskIF> // "tasks" data from API
// *** TODO: declare a type for pendingsList
pendingsList: Array<any> // pendings list from bootstrap filing
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I'll declare a type for this in my next PR as I keep working on the ticket.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Sounds good

Vue.use(Vuetify)
const vuetify = new Vuetify({})

// *** TODO: implement this
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I'll do this in my next PR as I keep working on the ticket.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Left side is code taken from TodoList.vue. Right side is modeled after FilingHistoryList.vue. Way cleaner!

(To be transparent, different sub-components/files handle the decoding and display of each filing type.)

Copy link
Collaborator

Choose a reason for hiding this comment

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

I agree :)

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

This is the template for pending items.

Actual items (eg, ContinuationIn.vue, below) use this template and can override any of the slots.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Here's a default (fallback) filing that uses the template and doesn't override any slots.

@severinbeauvais
Copy link
Collaborator Author

/gcbrun

@bcgov bcgov deleted a comment from bcregistry-sre Jul 17, 2024
@bcregistry-sre
Copy link
Collaborator

bcregistry-sre commented Jul 17, 2024

Copy link
Collaborator

@ketaki-deodhar ketaki-deodhar left a comment

Choose a reason for hiding this comment

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

Look good Sev!

@bcgov bcgov deleted a comment from bcregistry-sre Jul 17, 2024
@severinbeauvais severinbeauvais merged commit 9741518 into bcgov:main Jul 17, 2024
5 checks passed
severinbeauvais added a commit to severinbeauvais/business-filings-ui that referenced this pull request Jul 19, 2024
* - app version = 7.3.14
- now store pending bootstrap item to Pendings list
- added "pendings" store state, action, getter
- added pending conditionals to dashboard v-cards
- refactored PendingList.vue
- implemented PendingTemplate.vue
- implemented ContinuationIn.vue
- misc cleanup

* - renamed setPendings -> setPendingsList

* - added DefaultFiling.vue

* - renamed pendings -> pendingsList

---------

Co-authored-by: Severin Beauvais <severin.beauvais@gov.bc.ca>

WIP

- removed some unused CSS
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.

4 participants