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

Added a method to convert block to text and fixed warning #754

Merged
merged 6 commits into from
Feb 6, 2025

Conversation

abhayymishraa
Copy link
Collaborator

Resolves #750

Add the PR description here.

  • Added a utility function to convert block into text
  • Included emojis, /n
  • Created a extra method to convert to better viewing/
  • Added text field for all slack commands and tested for all commands

Screenshot

Response text from client.chat_postMessage

Screenshot from 2025-02-05 16-51-29

Reaponse.data looks like this

Screenshot from 2025-02-05 16-52-05

@arkid15r arkid15r marked this pull request as draft February 5, 2025 16:37
@abhayymishraa
Copy link
Collaborator Author

  • Added testcas for utils
    image

@abhayymishraa abhayymishraa marked this pull request as ready for review February 5, 2025 18:14
Copy link
Collaborator

@arkid15r arkid15r left a comment

Choose a reason for hiding this comment

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

LGTM 👍

What to improve:

  • better naming
  • never + strings in Python

@arkid15r arkid15r enabled auto-merge February 6, 2025 01:23
@arkid15r arkid15r added this pull request to the merge queue Feb 6, 2025
Merged via the queue into OWASP:main with commit 67bd336 Feb 6, 2025
13 checks passed
@abhayymishraa
Copy link
Collaborator Author

Thank You!!

@abhayymishraa abhayymishraa deleted the feat/unified-block-to-text branch February 6, 2025 01:45
Dishant1804 pushed a commit to Dishant1804/Nest that referenced this pull request Feb 7, 2025
* Added a method to convert block to text

* Added testcase for utils

* Update code

---------

Co-authored-by: Arkadii Yakovets <[email protected]>
Co-authored-by: Arkadii Yakovets <[email protected]>
github-merge-queue bot pushed a commit that referenced this pull request Feb 7, 2025
* anchor tags migrated

* Buttons migrated to chakraUI

* migarted buttons tooltip and anchortags to chakraUI

* fixed focus ring issue

* Generic component for details page (#678)

* base_design

* Add types and apply corrections.

* Improvement

* cleanup

* added issue label

* fixes

* pre-commit

---------

Co-authored-by: Kate Golovanova <[email protected]>

* Addition of Contribute Section in Home of Slack NestBot (#593)

* Addition of Contribute Section in Home of Slack NestBot

* Update contribute.py

* Update of apps/slack/commands/contribute

* Minor changes

* Update of code after the review

* Update backend/apps/slack/commands/contribute.py

Co-authored-by: Nitin Awari <[email protected]>

* Update urls.py

* Update contribute.py

* Update code

---------

Co-authored-by: Nitin Awari <[email protected]>
Co-authored-by: Arkadii Yakovets <[email protected]>
Co-authored-by: Arkadii Yakovets <[email protected]>

* resolved conflicts

* sort dropdown tooltip position modified

* test utils warpper used

* checks and format

* removed unnecessary button props

* removed redundant lines

* Update FE dependencies

* tooltip-bug (#744)

* Addition of website attribute to project schema (#740)

* Addition of website attribute to project schema

* Update vite.config.ts

* Update website-empty.yaml

* Update website-none.yaml

* Update website-none.yaml

* Update website-empty.yaml

* Update website-none.yaml

* Update website-empty.yaml

* Update website-empty.yaml

* Update project_test.py

* Update project.json

* Delete schema/tests/data/project/negative/website-empty.yaml

* Update code

---------

Co-authored-by: Arkadii Yakovets <[email protected]>
Co-authored-by: Arkadii Yakovets <[email protected]>

* Add blog to project schema (#745)

* added blog attribute to project schema

* added blog attribute

* Update project_test.py

* Update code

* Update code

---------

Co-authored-by: Arkadii Yakovets <[email protected]>

* Initialize OWASP chapter schema

* Refactor index synonyms (#748)

* Changes5

* Refactor index synonyms

* Update code

---------

Co-authored-by: Raghav Gupta 2 <[email protected]>
Co-authored-by: Arkadii Yakovets <[email protected]>

* write repository schema and tests (#741)

* write repository schema and tests

* fixed repository schema tests

* Update code

---------

Co-authored-by: Arkadii Yakovets <[email protected]>

* Fetch /gsoc command data from Algolia instead of DB (#712)

* gsoc command data fetch from algolia

* pre-commit and format checks

* moved get_gsoc_projects to search project file

* Update code

---------

Co-authored-by: Arkadii Yakovets <[email protected]>
Co-authored-by: Arkadii Yakovets <[email protected]>

* Reconfigure spell check

* Bump @chakra-ui/react from 3.6.0 to 3.7.0 in /frontend (#752)

Bumps [@chakra-ui/react](https://github.com/chakra-ui/chakra-ui/tree/HEAD/packages/react) from 3.6.0 to 3.7.0.
- [Release notes](https://github.com/chakra-ui/chakra-ui/releases)
- [Changelog](https://github.com/chakra-ui/chakra-ui/blob/main/packages/react/CHANGELOG.md)
- [Commits](https://github.com/chakra-ui/chakra-ui/commits/@chakra-ui/[email protected]/packages/react)

---
updated-dependencies:
- dependency-name: "@chakra-ui/react"
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Update params mapping

* Added a method to convert block to text and fixed warning  (#754)

* Added a method to convert block to text

* Added testcase for utils

* Update code

---------

Co-authored-by: Arkadii Yakovets <[email protected]>
Co-authored-by: Arkadii Yakovets <[email protected]>

* Update frontend dependencies

* Run poetry update

* conflicts resolved

* Cleanup: removing unused dependencies n dev dependencies (#601)

* "Cleanup: removing unused dependencies n dev dependencies "

* resolved conflicts

* Fix staging image build

* "Cleanup: removing unused dependencies n dev dependencies "

* fix run npm install install prettier plugin tailwindcss

* fix run run npm lint check

* fix check for uncommited changes

* resolved conflicts  n npm format check

* "Cleanup: removing unused dependencies n dev dependencies "

* resolved conflicts

* Fix staging image build

* "Cleanup: removing unused dependencies n dev dependencies "

* fix run npm install install prettier plugin tailwindcss

* fix run run npm lint check

* fix check for uncommited changes

* resolved conflicts  n npm format check

* updated one

* updated fix

* removed unwanted files from updated fix

* Update code

---------

Co-authored-by: Arkadii Yakovets <[email protected]>

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: Kate Golovanova <[email protected]>
Co-authored-by: Raj gupta <[email protected]>
Co-authored-by: AbhayTopno <[email protected]>
Co-authored-by: Nitin Awari <[email protected]>
Co-authored-by: Arkadii Yakovets <[email protected]>
Co-authored-by: Arkadii Yakovets <[email protected]>
Co-authored-by: yashgoyal0110 <[email protected]>
Co-authored-by: Rusheek <[email protected]>
Co-authored-by: Raghavg18 <[email protected]>
Co-authored-by: Raghav Gupta 2 <[email protected]>
Co-authored-by: Shyam Kumar <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Abhay <[email protected]>
Co-authored-by: Diamond Girl <[email protected]>
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.

Add unified block-to-text converter utility for Slack messages
2 participants