Skip to content

Conversation

hardchor
Copy link

Fixes #56

@rbiseck3
Copy link
Contributor

Can this also get an update to the changelog? That'll prevent the CI linters from passing.

@hardchor
Copy link
Author

@rbiseck3 done - I tried to follow the existing format as closely as possible

@rbiseck3
Copy link
Contributor

Sorry, I think the changelog needs to be updated one more time to pull in changes that had since been merged in.

### Fixes

* **Update Sharepoint Creds and Expected docs**
- **Update Sharepoint Creds and Expected docs**
Copy link
Contributor

Choose a reason for hiding this comment

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

Looks like something updated all of these, can we limit the changes to the changelog in this PR?

@potter-potter
Copy link
Contributor

@hardchor Thanks for this. I'll just put it in a new PR since we have to update some of the ingest tests, etc. Will just be easier. Thanks so much for working on this!

See #171

@rbiseck3
Copy link
Contributor

Closing in favor of new PR: #171

@rbiseck3 rbiseck3 closed this Oct 15, 2024
@hardchor hardchor deleted the notion-in-trash-property branch October 31, 2024 15:51
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.

bug/notion ingestion not working
3 participants