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

Change minimalFieldMapping.uid in brh to _unique_id #2323

Merged
merged 10 commits into from
Jan 26, 2023
Merged

Change minimalFieldMapping.uid in brh to _unique_id #2323

merged 10 commits into from
Jan 26, 2023

Conversation

nss10
Copy link
Contributor

@nss10 nss10 commented Jan 19, 2023 •
edited by jira bot
Loading

Link to Jira ticket if there is one: BRH-389

Environments

  • qa-brh

Description of changes

  • Updating the minimalFieldMapping.uid portal config in qa-brh from frontend_uid to _unique_id
  • Pointing data portal to fix_datarow-key-issue branch to include the changes made in this data-portal PR

@nss10 nss10 requested a review from haraprasadj as a code owner January 19, 2023 03:43
@PlanXCyborg
Copy link
Contributor

qa-brh.planx-pla.net/manifest.json

⚠️ Services on branch

  • portal

@nss10 nss10 removed the jenkins-dcp test label Jan 19, 2023
@nss10 nss10 added the jenkins-dcp test label Jan 19, 2023
PlanXCyborg and others added 2 commits January 20, 2023 13:07
@PlanXCyborg @haraprasadj @nss10
Gen3 Monthly Release 2023.01 QA-BRH 2023.01 qa-brh.planx-pla.net 1673…
ca15d2c 
…984591 (#2320)

* apply version 2023.01 to qa-brh.planx-pla.net

* apply version 2023.01 to qa-brh.planx-pla.net

* Update manifest.json

Co-authored-by: Hara Prasad <[email protected]>
@nss10
Merge branch 'master' into fix/data-row-key-update
cbdc0c1
@nss10 nss10 changed the title Change datarow-key Change minimalFieldMapping.uid in brh to _unique_id Jan 20, 2023
@PlanXCyborg
Copy link
Contributor

qa-brh.planx-pla.net/manifest.json

⚠️ Services are being downgraded

  • portal

@PlanXCyborg
Copy link
Contributor

qa-brh.planx-pla.net/manifest.json

Deployment changes

  • portal
    • New portal config options to exclude File nodes from the charts added: use "graphql.chartNodesExcludeFiles" to exclude File nodes from graphql chart, or use "components.index.homepageChartNodesExcludeFiles" to exclude File nodes from homepage chart. Note both of them are default to false so you have to explicitly set them to true to enable them (PXP-10565/PPS-177 fix: exclude file nodes from chart with options data-portal#1208)

1 similar comment
@PlanXCyborg
Copy link
Contributor

qa-brh.planx-pla.net/manifest.json

Deployment changes

  • portal
    • New portal config options to exclude File nodes from the charts added: use "graphql.chartNodesExcludeFiles" to exclude File nodes from graphql chart, or use "components.index.homepageChartNodesExcludeFiles" to exclude File nodes from homepage chart. Note both of them are default to false so you have to explicitly set them to true to enable them (PXP-10565/PPS-177 fix: exclude file nodes from chart with options data-portal#1208)

@nss10 nss10 merged commit a2d55a3 into master Jan 26, 2023
@nss10 nss10 deleted the fix/data-row-key-update branch January 26, 2023 17:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@haraprasadj haraprasadj haraprasadj approved these changes

Assignees
No one assigned
Labels
test-portal-discoveryPageTest
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@nss10 @PlanXCyborg @haraprasadj