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

Add Organization Rate #171

Open
wants to merge 8 commits into
base: ab_migrate
Choose a base branch
from
Open

Add Organization Rate #171

wants to merge 8 commits into from

Conversation

luthian
Copy link
Contributor

@luthian luthian commented Sep 20, 2023

This PR adds the OrganizationRate to the Organization class as well as adding custom headers and slots to IFXOrganizationList and a slot toIFXOrganizationDetail. This allows a calling page, such as one from CBSN, to add content to these pages which they can use to display and attach additional data to the Organization object.

The first use of this will be CBSN adding OrganizationRates.

luthian and others added 7 commits September 14, 2023 15:11
- Also comment out a console.log
…ntent

- The headers default to the existing ones
- We now re-emit the `selected` changes so the parent can use them
- There is a slot for custom buttons and a slot for custom content
- This is used by CBSN for organization rate setting.
- In this case, CBSN Organization Rate
- Has to be kabob case
@luthian luthian changed the title Add Organization Rate WIP: Add Organization Rate Sep 28, 2023
- Also add decompose to `organizationRate`
@luthian luthian changed the title WIP: Add Organization Rate Add Organization Rate Oct 4, 2023
@@ -400,6 +406,12 @@ export default class IFXAPIService {
})
newOrgData.users = organizationUserDataObjs
}

// Check if incoming orgData has rates
if (orgData.organization_rates && orgData.organization_rates.length) {
Copy link
Collaborator

Choose a reason for hiding this comment

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

Shouldn't this be organizationRates?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I don't think so since this data is either coming from the server or being decomposed and, hence, is part of the data object.

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