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

Update Goldman Sachs funds (formerly NN funds) #40

Merged
merged 1 commit into from
Dec 12, 2023

Conversation

jakedane
Copy link
Contributor

NN was acquired by Goldman Sachs. This change updates the asset manager logo, the fund names and the fund symbols as the NN name is no longer used by Goldman Sachs.

This change also corrects the KIID download link and updates the TER and internal transaction costs as per costs shown for these funds on: https://www.gsam.com/responsible-investing/nl-NL/professional/funds

NN was acquired by Goldman Sachs. This change updates the asset manager
logo, the fund names and the fund symbols as the NN name is no longer
used by Goldman Sachs.

This change also corrects the KIID download link and updates the TER and
internal transaction costs as per costs shown for these funds on:
https://www.gsam.com/responsible-investing/nl-NL/professional/funds
@nicwortel
Copy link
Owner

Thank you for the awesome pull request! 👍 I wasn't aware of the change.

@nicwortel nicwortel merged commit 70ae3e2 into nicwortel:master Dec 12, 2023
3 checks passed
@jakedane jakedane deleted the gsam branch December 12, 2023 13:05
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