Skip to content

docs: add FactSet Enterprise connector docs #588

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

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

eschmidtfds
Copy link

This PR adds docs for the new combined FactSet connector.
This is replacing the exisint FactSet Analytics (reused, renamed) and RMS (will be deprecated) connectors.

@prmerger-automator prmerger-automator bot added the do-not-merge Don't merge this yet label Jun 30, 2025
Copy link
Contributor

Learn Build status updates of commit 49bdabb:

⚠️ Validation status: warnings

File Status Preview URL Details
powerquery-docs/connectors/factset.md ⚠️Warning Details
powerquery-docs/connectors/index.md ✅Succeeded
powerquery-docs/connectors/media/factset/authentication.png ✅Succeeded
powerquery-docs/connectors/media/factset/connect.png ✅Succeeded
powerquery-docs/connectors/media/factset/navigator.png ✅Succeeded
powerquery-docs/connectors/media/factset/open-power-bi.png ✅Succeeded
powerquery-docs/toc.yml ✅Succeeded

powerquery-docs/connectors/factset.md

  • Line 1, Column 1: [Warning: author-missing - See documentation] Missing required attribute: 'author'. Add the current author's GitHub ID.
  • Line 1, Column 1: [Warning: title-missing - See documentation] Missing required attribute: 'title'. Add a title string to show in search engine results.
  • Line 1, Column 1: [Warning: ms-author-missing - See documentation] Missing required attribute: 'ms.author'. Add the current author's Microsoft alias.
  • Line 0, Column 0: [Suggestion: filename-incomplete - See documentation] The filename contains an incomplete word, misspelled word, acronym, or abbreviation that is disallowed.
  • Line 1, Column 1: [Suggestion: ms-date-missing - See documentation] Missing required attribute: 'ms.date'.
  • Line 1, Column 1: [Suggestion: description-missing - See documentation] Missing required attribute: 'description'.
  • Line 46, Column 5: [Suggestion: alt-text-bad-value - See documentation] Alt text 'Authentication' is the same as the image file name. This redundancy adds no value for users of screen readers. For accessibility, add more descriptive alt text.
  • Line 52, Column 5: [Suggestion: alt-text-bad-value - See documentation] Alt text 'Navigator' is the same as the image file name. This redundancy adds no value for users of screen readers. For accessibility, add more descriptive alt text.

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

@eschmidtfds eschmidtfds changed the title docs: add factset enterprise connector docs docs: add FactSet Enterprise connector docs Jun 30, 2025
@eschmidtfds
Copy link
Author

@microsoft-github-policy-service agree company="FactSet Research Systems"

Copy link
Contributor

@eschmidtfds : Thanks for your contribution! The author(s) and reviewer(s) have been notified to review your proposed change.

@eschmidtfds eschmidtfds force-pushed the connectors/add-factset branch from 49bdabb to faffdf5 Compare June 30, 2025 19:57
Copy link
Contributor

Learn Build status updates of commit faffdf5:

💡 Validation status: suggestions

File Status Preview URL Details
powerquery-docs/connectors/factset.md 💡Suggestion Details
powerquery-docs/connectors/index.md ✅Succeeded
powerquery-docs/connectors/media/factset/authentication.png ✅Succeeded
powerquery-docs/connectors/media/factset/connect.png ✅Succeeded
powerquery-docs/connectors/media/factset/navigator.png ✅Succeeded
powerquery-docs/connectors/media/factset/open-power-bi.png ✅Succeeded
powerquery-docs/toc.yml ✅Succeeded

powerquery-docs/connectors/factset.md

  • Line 0, Column 0: [Suggestion: filename-incomplete - See documentation] The filename contains an incomplete word, misspelled word, acronym, or abbreviation that is disallowed.
  • Line 8, Column 12: [Suggestion: ms-author-invalid - See documentation] Invalid value for 'ms.author', 'unknown' is not a valid Microsoft alias.

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

Copy link
Contributor

PRMerger Results

Issue Description
Added File(s) This PR contains added files. New files require human review.
Yaml File(s) This PR includes changes to .yml file(s) owned by another author.
File Change Percent This PR contains file(s) with more than 30% file change.
Image File This PR added or updated an image file(s).
Index File This PR contains a change to the index file.

@eschmidtfds
Copy link
Author

Hello! I wasn't quite sure how to fill in the author and ms.author fields. Please let me know what values I should fill in.

Copy link
Contributor

Learn Build status updates of commit 44cfcfb:

💡 Validation status: suggestions

File Status Preview URL Details
powerquery-docs/connectors/factset.md 💡Suggestion Details
powerquery-docs/connectors/index.md ✅Succeeded
powerquery-docs/connectors/media/factset/authentication.png ✅Succeeded
powerquery-docs/connectors/media/factset/connect.png ✅Succeeded
powerquery-docs/connectors/media/factset/navigator.png ✅Succeeded
powerquery-docs/connectors/media/factset/open-power-bi.png ✅Succeeded
powerquery-docs/toc.yml ✅Succeeded

powerquery-docs/connectors/factset.md

  • Line 0, Column 0: [Suggestion: filename-incomplete - See documentation] The filename contains an incomplete word, misspelled word, acronym, or abbreviation that is disallowed.
  • Line 8, Column 12: [Suggestion: ms-author-invalid - See documentation] Invalid value for 'ms.author', 'unknown' is not a valid Microsoft alias.

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

Copy link
Contributor

PRMerger Results

Issue Description
Added File(s) This PR contains added files. New files require human review.
Yaml File(s) This PR includes changes to .yml file(s) owned by another author.
File Change Percent This PR contains file(s) with more than 30% file change.
Image File This PR added or updated an image file(s).
Index File This PR contains a change to the index file.

@DougKlopfenstein
Copy link
Collaborator

Hi @eschmidtfds - The author metadata should be filled in with your GitHub alias (eschmidtfds). You can set the ms.author metadata to my Microsoft alias (dougklo).

@v-dirichards
Copy link
Contributor

@eschmidtfds Thanks for your pull request. A member of the content team will review it.

Copy link
Contributor

Learn Build status updates of commit cc17b2d:

💡 Validation status: suggestions

File Status Preview URL Details
powerquery-docs/connectors/factset.md 💡Suggestion Details
powerquery-docs/connectors/index.md ✅Succeeded
powerquery-docs/connectors/media/factset/authentication.png ✅Succeeded
powerquery-docs/connectors/media/factset/connect.png ✅Succeeded
powerquery-docs/connectors/media/factset/navigator.png ✅Succeeded
powerquery-docs/connectors/media/factset/open-power-bi.png ✅Succeeded
powerquery-docs/toc.yml ✅Succeeded

powerquery-docs/connectors/factset.md

  • Line 0, Column 0: [Suggestion: filename-incomplete - See documentation] The filename contains an incomplete word, misspelled word, acronym, or abbreviation that is disallowed.

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

Copy link
Contributor

PRMerger Results

Issue Description
Added File(s) This PR contains added files. New files require human review.
Yaml File(s) This PR includes changes to .yml file(s) owned by another author.
File Change Percent This PR contains file(s) with more than 30% file change.
Image File This PR added or updated an image file(s).
Index File This PR contains a change to the index file.

@eschmidtfds
Copy link
Author

Hi @DougKlopfenstein done!

@v-dirichards
Copy link
Contributor

@DougKlopfenstein for review, thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants