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

Funder and Recipient GrantNav export: #110

Merged
merged 3 commits into from
Mar 10, 2022
Merged

Funder and Recipient GrantNav export: #110

merged 3 commits into from
Mar 10, 2022

Conversation

kindly
Copy link
Contributor

@kindly kindly commented Feb 3, 2022

Uses orginfocache to get a list of all related orgids and all org names.

It gathers statistics about the orgs including amount statististics
broken down by currency.

Needs a refresh of org cache to work.

@kindly kindly force-pushed the org-output branch 5 times, most recently from 2be47a5 to b677dc1 Compare February 3, 2022 17:16
Copy link
Member

@michaelwood michaelwood left a comment

Choose a reason for hiding this comment

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

1 Q on this.

Uses orginfocache to get a list of all related orgids and all org names.

It gathers statistics about the orgs including amount statististics
broken down by currency.
Rename to tmp_related_orgids so that it is indicated to public that this
isn't a table that should be relied on to always exist.

Also fixes white space issues in SQL
@michaelwood
Copy link
Member

@kindly "Needs a refresh of org cache to work." does this mean that the org cache needs to be populated rather than it needs reloading. I.e. do I need to do anything on the live datastore after deploying this?

@kindly
Copy link
Contributor Author

kindly commented Mar 9, 2022

It needs a complete reload, unless it was done recently. This is because the charity finder data has new information in that we need.

@michaelwood michaelwood merged commit 039c6d2 into master Mar 10, 2022
@michaelwood michaelwood deleted the org-output branch March 10, 2022 11:08
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.

None yet

3 participants