Skip to content
This repository has been archived by the owner on Apr 18, 2024. It is now read-only.

feat: DIA-820: [FE] Make List type columns display as intended type with out the ability for users to change type for Data Discovery #289

Merged
merged 3 commits into from
Jan 17, 2024

Conversation

yyassi-heartex
Copy link
Contributor

@yyassi-heartex yyassi-heartex commented Jan 12, 2024

PR fulfills these requirements

  • Commit message(s) and PR title follows the format [fix|feat|ci|chore|doc]: TICKET-ID: Short description of change made ex. fix: DEV-XXXX: Removed inconsistent code usage causing intermittent errors
  • Tests for the changes have been added/updated (for bug fixes/features)
  • Docs have been added/updated (for bug fixes/features)
  • Best efforts were made to ensure docs/code are concise and coherent (checked for spelling/grammatical errors, commented out code, debug logs etc.)
  • Self-reviewed and ran all changes on a local instance (for bug fixes/features)

Change has impacts in these area(s)

(check all that apply)

  • Product design
  • Frontend

Describe the reason for change

For purposes of Data Explorer column type selection is disallowed, Projects will work as before

…ith out the ability for users to change type for Data Discovery
@yyassi-heartex yyassi-heartex merged commit 18fac7b into master Jan 17, 2024
5 checks passed
@yyassi-heartex yyassi-heartex deleted the fb-dia-820/type-selection branch January 17, 2024 13:26
juliosgarbi pushed a commit that referenced this pull request Jan 22, 2024
…ith out the ability for users to change type for Data Discovery (#289)
juliosgarbi added a commit that referenced this pull request Jan 22, 2024
feat: DIA-820: [FE] Make List type columns display as intended type with out the ability for users to change type for Data Discovery (#289)

Co-authored-by: yyassi-heartex <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants