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

tag-functionality-added #947

Closed

Conversation

SURAJ-SHARMA27
Copy link

Description

This pull request addresses the previous limitation in the organization pages where the "Select Tags" section lacked functionality. Specifically, the "+" button was non-functional, and only two tags ("python" and "javascript") were hardcoded. The changes include introducing a new field, "org_tags," to the organization schema. This addition enables users to customize and add tags to organizations dynamically. A search functionality has been implemented, allowing users to type and receive related tags in a dropdown. The selected tags are reflected on the organization feeds page upon publishing.

Related Issue

#946 fixed

Motivation and Context

This change is necessary to enhance the user experience on organization pages. Previously, users were limited in their ability to customize tags for organizations. The introduction of the "org_tags" field and the dynamic tagging functionality solves this problem, providing users with the flexibility to add, remove, and search for tags based on their preferences.

Screenshots or GIF (In case of UI changes):

before:

tag-issue.mp4

after:

tag-solution.mp4

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@SURAJ-SHARMA27
Copy link
Author

@shivareddy6 could you review the PR and the related issue?

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.

1 participant