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

[FEATURE] Slack notification message - remove duplicated link text, decluttering #10890

Open
wants to merge 14 commits into
base: develop
Choose a base branch
from

Conversation

data-han
Copy link

@data-han data-han commented Jan 23, 2025

These issues have been affecting me for quite some time. While they were said to be resolved, I’m still encountering them even after 1–2+ years, since version 0.15.x. As a result, I’ve started testing the rendering to see if it might help resolve my problems. Ultimately, I’d prefer a change that allows me to directly pull the original GX into a Docker image, rather than relying on a forked repository.

Links to previous issues:

Changes done:

  • Remove duplicated link text ("file://") that is below "Validation Results" and above "Asset/ Expectation Suite"
  • Markdown highlight the actual Asset and Expectation Suite such that it's easier to see when you have many slack notifications
  • Add back Summary suite (number of expectations out of total met) --> I find this rather useful as this gives a gauge of how many are met at one glance, is it a BIG or SMALL issue.
  • Add next line to each of these items

What it was before:
image

What it is now for me:
image

  • [x ] PR title is prefixed with one of: [BUGFIX], [FEATURE], [DOCS], [MAINTENANCE], [CONTRIB]
  • [x ] Code is linted - run invoke lint (uses ruff format + ruff check)
  • Appropriate tests and docs have been updated

For more information about contributing, visit our community resources.

After you submit your PR, keep the page open and monitor the statuses of the various checks made by our continuous integration process at the bottom of the page. Please fix any issues that come up and reach out on Slack if you need help. Thanks for contributing!

Copy link

netlify bot commented Jan 23, 2025

👷 Deploy request for niobium-lead-7998 pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit 2506b2b

@gx-cla-bot
Copy link

gx-cla-bot bot commented Jan 23, 2025

A new contributor, HUZZAH! Welcome and thanks for joining our community. In order to accept a pull request we require that all contributors sign our Contributor License Agreement. We have two different CLAs, depending on whether you are contributing to GX in a personal or professional capacity. Please sign the one that is applicable to your situation so that we may accept your contribution:

Individual Contributor License Agreement v1.0
Software Grant and Corporate Contributor License Agreement v1.0

Once you have signed the CLA, you can add a comment with the text @cla-bot check and the bot will update the PR status!

Please reach out to the #gx-community-support channel, on our Slack if you have any questions or if you have already signed the CLA and are receiving this message in error.

Users missing a CLA: [email protected]

@gx-cla-bot
Copy link

gx-cla-bot bot commented Jan 23, 2025

A new contributor, HUZZAH! Welcome and thanks for joining our community. In order to accept a pull request we require that all contributors sign our Contributor License Agreement. We have two different CLAs, depending on whether you are contributing to GX in a personal or professional capacity. Please sign the one that is applicable to your situation so that we may accept your contribution:

Individual Contributor License Agreement v1.0
Software Grant and Corporate Contributor License Agreement v1.0

Once you have signed the CLA, you can add a comment with the text @cla-bot check and the bot will update the PR status!

Please reach out to the #gx-community-support channel, on our Slack if you have any questions or if you have already signed the CLA and are receiving this message in error.

Users missing a CLA: [email protected]

@gx-cla-bot
Copy link

gx-cla-bot bot commented Jan 23, 2025

A new contributor, HUZZAH! Welcome and thanks for joining our community. In order to accept a pull request we require that all contributors sign our Contributor License Agreement. We have two different CLAs, depending on whether you are contributing to GX in a personal or professional capacity. Please sign the one that is applicable to your situation so that we may accept your contribution:

Individual Contributor License Agreement v1.0
Software Grant and Corporate Contributor License Agreement v1.0

Once you have signed the CLA, you can add a comment with the text @cla-bot check and the bot will update the PR status!

Please reach out to the #gx-community-support channel, on our Slack if you have any questions or if you have already signed the CLA and are receiving this message in error.

Users missing a CLA: [email protected]

@gx-cla-bot
Copy link

gx-cla-bot bot commented Jan 23, 2025

A new contributor, HUZZAH! Welcome and thanks for joining our community. In order to accept a pull request we require that all contributors sign our Contributor License Agreement. We have two different CLAs, depending on whether you are contributing to GX in a personal or professional capacity. Please sign the one that is applicable to your situation so that we may accept your contribution:

Individual Contributor License Agreement v1.0
Software Grant and Corporate Contributor License Agreement v1.0

Once you have signed the CLA, you can add a comment with the text @cla-bot check and the bot will update the PR status!

Please reach out to the #gx-community-support channel, on our Slack if you have any questions or if you have already signed the CLA and are receiving this message in error.

Users missing a CLA: [email protected]

@gx-cla-bot
Copy link

gx-cla-bot bot commented Jan 23, 2025

A new contributor, HUZZAH! Welcome and thanks for joining our community. In order to accept a pull request we require that all contributors sign our Contributor License Agreement. We have two different CLAs, depending on whether you are contributing to GX in a personal or professional capacity. Please sign the one that is applicable to your situation so that we may accept your contribution:

Individual Contributor License Agreement v1.0
Software Grant and Corporate Contributor License Agreement v1.0

Once you have signed the CLA, you can add a comment with the text @cla-bot check and the bot will update the PR status!

Please reach out to the #gx-community-support channel, on our Slack if you have any questions or if you have already signed the CLA and are receiving this message in error.

Users missing a CLA: [email protected]

@gx-cla-bot
Copy link

gx-cla-bot bot commented Jan 23, 2025

A new contributor, HUZZAH! Welcome and thanks for joining our community. In order to accept a pull request we require that all contributors sign our Contributor License Agreement. We have two different CLAs, depending on whether you are contributing to GX in a personal or professional capacity. Please sign the one that is applicable to your situation so that we may accept your contribution:

Individual Contributor License Agreement v1.0
Software Grant and Corporate Contributor License Agreement v1.0

Once you have signed the CLA, you can add a comment with the text @cla-bot check and the bot will update the PR status!

Please reach out to the #gx-community-support channel, on our Slack if you have any questions or if you have already signed the CLA and are receiving this message in error.

Users missing a CLA: [email protected]

@gx-cla-bot
Copy link

gx-cla-bot bot commented Jan 23, 2025

A new contributor, HUZZAH! Welcome and thanks for joining our community. In order to accept a pull request we require that all contributors sign our Contributor License Agreement. We have two different CLAs, depending on whether you are contributing to GX in a personal or professional capacity. Please sign the one that is applicable to your situation so that we may accept your contribution:

Individual Contributor License Agreement v1.0
Software Grant and Corporate Contributor License Agreement v1.0

Once you have signed the CLA, you can add a comment with the text @cla-bot check and the bot will update the PR status!

Please reach out to the #gx-community-support channel, on our Slack if you have any questions or if you have already signed the CLA and are receiving this message in error.

Users missing a CLA: [email protected]

@data-han
Copy link
Author

@cla-bot check

@gx-cla-bot
Copy link

gx-cla-bot bot commented Jan 23, 2025

A new contributor, HUZZAH! Welcome and thanks for joining our community. In order to accept a pull request we require that all contributors sign our Contributor License Agreement. We have two different CLAs, depending on whether you are contributing to GX in a personal or professional capacity. Please sign the one that is applicable to your situation so that we may accept your contribution:

Individual Contributor License Agreement v1.0
Software Grant and Corporate Contributor License Agreement v1.0

Once you have signed the CLA, you can add a comment with the text @cla-bot check and the bot will update the PR status!

Please reach out to the #gx-community-support channel, on our Slack if you have any questions or if you have already signed the CLA and are receiving this message in error.

Users missing a CLA: [email protected]

@data-han
Copy link
Author

@adeola-ak Hi Adeola, can i check how do i get my CLA approved ? I added the email

@adeola-ak
Copy link
Contributor

hi @data-han thanks for the submission. I believe we were having an issue with the signing and may need to make a copy of your PR once it's been reviewed and approved to be accepted. I'll look into this now -- thanks again

@NathanFarmer NathanFarmer self-assigned this Jan 24, 2025
@gx-cla-bot
Copy link

gx-cla-bot bot commented Jan 28, 2025

A new contributor, HUZZAH! Welcome and thanks for joining our community. In order to accept a pull request we require that all contributors sign our Contributor License Agreement. We have two different CLAs, depending on whether you are contributing to GX in a personal or professional capacity. Please sign the one that is applicable to your situation so that we may accept your contribution:

Individual Contributor License Agreement v1.0
Software Grant and Corporate Contributor License Agreement v1.0

Once you have signed the CLA, you can add a comment with the text @cla-bot check and the bot will update the PR status!

Please reach out to the #gx-community-support channel, on our Slack if you have any questions or if you have already signed the CLA and are receiving this message in error.

Users missing a CLA: [email protected]

Copy link
Contributor

@NathanFarmer NathanFarmer left a comment

Choose a reason for hiding this comment

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

@data-han Thanks for the contribution! I needed to approve to run the tests in CI and it looks like there are some failures. I also had some comments in your PR.

Also, it looks like for the cla-bot you signed with a different email address than the one shown in the message. Can you please sign with [email protected]?

Comment on lines -54 to -56
status = "Failed :x:"
if validation_result.success:
status = "Success :tada:"
Copy link
Contributor

Choose a reason for hiding this comment

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

Was this 🎉 no longer showing in the notification?

Copy link
Author

Choose a reason for hiding this comment

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

I was referencing to this PR (#10129) and I thought it wasn't intended as the status will be shown in the header too

image

I can add it back if that was the original intention; at which line should I add it back since I removed the entire chunk of duplicated lines (including this status)

great_expectations/render/renderer/slack_renderer.py Outdated Show resolved Hide resolved
@gx-cla-bot
Copy link

gx-cla-bot bot commented Jan 29, 2025

A new contributor, HUZZAH! Welcome and thanks for joining our community. In order to accept a pull request we require that all contributors sign our Contributor License Agreement. We have two different CLAs, depending on whether you are contributing to GX in a personal or professional capacity. Please sign the one that is applicable to your situation so that we may accept your contribution:

Individual Contributor License Agreement v1.0
Software Grant and Corporate Contributor License Agreement v1.0

Once you have signed the CLA, you can add a comment with the text @cla-bot check and the bot will update the PR status!

Please reach out to the #gx-community-support channel, on our Slack if you have any questions or if you have already signed the CLA and are receiving this message in error.

Users missing a CLA: [email protected]

@data-han
Copy link
Author

@NathanFarmer Thank you for taking a look at my PR, appreciate it!

As for your comments

  • missing CLA --> I've already signed with the email before, but I just did again. Not sure if this is registered
  • Capitalized 'Expectations'
  • Left a comment for the status icon; awaiting your feedback
  • Failures in CI: could you help to see what error that is? I've seen the pytest but it looks to me that it was expecting certain message. It could be that I have already removed it (intended with this PR) and the pytest needs to be adjusted
image

@data-han data-han changed the title [FEATURE] Remove duplicated link text, markdown Asset and Suite [FEATURE] Slack notification message - remove duplicated link text, decluttering Feb 5, 2025
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