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.

Sign up for GitHub

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

Jump to bottom

enhance: add visibility to the Columns field #1390

Merged
merged 7 commits into from
Jan 24, 2024
Merged

enhance: add visibility to the Columns field #1390

merged 7 commits into from
Jan 24, 2024

Conversation

sapayth
Copy link
Member

@sapayth sapayth commented Sep 15, 2023

Add visibility to the Columns field to display or hide columns field based on some condition.

image

Fixes #521

@Rat01047
Copy link

Rat01047 commented Sep 19, 2023 •
edited
Loading

Issue-1:
Existing column fields > do not have Visibility option

Scenario:

  1. In develop-branch > Add Column field
  2. Switch to PR-branch > Edit previously added Column
  3. No reflection of PR-branch-changes ❌
  4. Add Column-field again > Edit > New changes present ✅

Screenshot for reference:
Column-added in develop-branch,
image

Column-added in pr-branch,
image

@Rat01047 Rat01047 added the bug label Sep 22, 2023
@Rat01047 Rat01047 added QA Approved This PR is approved by the QA team and removed bug QA In Progress labels Sep 26, 2023
@Rat01047 Rat01047 added QA In Progress and removed QA Approved This PR is approved by the QA team labels Jan 24, 2024
@Rat01047
Copy link

@sapayth
This issue is occurring again #1390 (comment)

@sapayth
Copy link
Member Author

sapayth commented Jan 24, 2024

@sapayth This issue is occurring again #1390 (comment)

Try running the upgrader @Rat01047 bhai

@Rat01047 Rat01047 added the bug label Jan 24, 2024
@Rat01047 Rat01047 added QA Approved This PR is approved by the QA team Ready to Merge This PR is now ready to be merged and removed bug QA In Progress labels Jan 24, 2024
@sapayth sapayth merged commit 180e0e9 into weDevsOfficial:develop Jan 24, 2024
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@Rat01047 Rat01047

Labels
QA Approved This PR is approved by the QA team Ready to Merge This PR is now ready to be merged
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@sapayth @Rat01047