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

fix: Update Readme Links #82

Merged
merged 2 commits into from
Feb 18, 2022
Merged

fix: Update Readme Links #82

merged 2 commits into from
Feb 18, 2022

Conversation

zngly-vlad
Copy link
Contributor

  • updated all instaces of morphir.zngly.com to resources.finos.com/morphir
  • This was the proposed destination of the resource hub by the ditto team and finos team

@mcleo-d

@netlify
Copy link

netlify bot commented Feb 17, 2022

✔️ Deploy Preview for angry-raman-7c44f6 canceled.

🔨 Explore the source changes: d171b65

🔍 Inspect the deploy log: https://app.netlify.com/sites/angry-raman-7c44f6/deploys/620e5179c0dfd1000728d74b

@mcleo-d
Copy link
Member

mcleo-d commented Feb 17, 2022

Hi @AttilaMihaly - It would be awesome if you could merge this pull request. Thanks 🚀

@stephengoldbaum stephengoldbaum merged commit 99d9613 into finos:main Feb 18, 2022
DamianReeves pushed a commit that referenced this pull request May 31, 2024
* Add generation of wrappers for keeping record references

Now we don't remove parameters representing records from functions .
This fixes problems where the a column may be lead to ambiguities.

* Code review suggestions
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.

3 participants