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

Adds data source properties to the Jira connector #1516

Merged
merged 21 commits into from
Oct 5, 2023

Conversation

rvztz
Copy link
Contributor

@rvztz rvztz commented Sep 25, 2023

Defines update_source_metadata for the delta_table connector.

CHANGELOG.md Outdated Show resolved Hide resolved
self.parsed_fields["updated"],
"%Y-%m-%dT%H:%M:%S.%f%z",
).isoformat(),
version=version,
Copy link
Contributor

Choose a reason for hiding this comment

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

I don't see a version in any of our expected outputs. how are we validating this?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It's not always set up, so in our tests this property is empty. The object definition (based on inspecting the response) can be found here.

Copy link
Contributor

Choose a reason for hiding this comment

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

got it, so in theory this could exist, but doesn't in our test content.

Copy link
Contributor

Choose a reason for hiding this comment

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

Is this field a "(~release) version" for a project, or a version for the issue content (say, before and after an edit to the issue content)?

Copy link
Contributor

@ahmetmeleq ahmetmeleq Sep 26, 2023

Choose a reason for hiding this comment

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

If the field is a version for the issue content itself, testing this might need updating documents at the source to introduce a second version to some issues

I think it'll be worth it to do those updates in our test jira org and test if it's working as intended (mainly since we're introducing the version field with this PR)

At best, it'll only require a change in a document in our test jira org and rerunning the CI. At worst, it will still save us some debugging and a patch later on

@rvztz rvztz marked this pull request as ready for review September 26, 2023 16:44
Copy link
Contributor

@ryannikolaidis ryannikolaidis left a comment

Choose a reason for hiding this comment

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

lgtm!

CHANGELOG.md Outdated
Comment on lines 30 to 31
* **Adds data source properties to SharePoint, Outlook, Onedrive, Reddit, Slack, and DeltaTable connectors** These properties (date_created, date_modified, version, source_url, record_locator) are written to element metadata during ingest, mapping elements to information about the document source from which they derive. This functionality enables downstream applications to reveal source document applications, e.g. a link to a GDrive doc, Salesforce record, etc.
* **Adds data source properties to SharePoint, Outlook, Onedrive, Reddit, Slack, DeltaTable and Jira connectors** These properties (date_created, date_modified, version, source_url, record_locator) are written to element metadata during ingest, mapping elements to information about the document source from which they derive. This functionality enables downstream applications to reveal source document applications, e.g. a link to a GDrive doc, Salesforce record, etc.
* **Add functionality to save embedded images in PDF's separately as images** This allows users to save embedded images in PDF's separately as images, given some directory path. The saved image path is written to the metadata for the Image element. Downstream applications may benefit by providing users with image links from relevant "hits."
Copy link
Contributor

Choose a reason for hiding this comment

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

no, we shouldn't change this one. these were previous version changes

@ryannikolaidis
Copy link
Contributor

ryannikolaidis commented Oct 4, 2023

@rvztz see comment on changelog.

@ryannikolaidis ryannikolaidis merged commit 6d8572d into main Oct 5, 2023
@ryannikolaidis ryannikolaidis deleted the rvztz/data-source-properties-jira branch October 5, 2023 03:04
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