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

ClientSideComponents fra portfolio-extensions er tenant-wide #1266

Closed
tarjeieo opened this issue Sep 12, 2023 · 6 comments
Closed

ClientSideComponents fra portfolio-extensions er tenant-wide #1266

tarjeieo opened this issue Sep 12, 2023 · 6 comments
Assignees
Labels
bug Something isn't working complexity: medium Issue/PR with medium complexity priority: high Issue with high priority
Milestone

Comments

@tarjeieo
Copy link
Member

tarjeieo commented Sep 12, 2023

Beskriv feilen

Clientsidekomponentene fra portfolio-extensions pakka er tenant-wide. Dette medfører at koden for IdeaRegistration, IdeaProcessing og IdeaProjectData kjører tenant-wide.

Dette er litt unødvendig i grunn.

Hvordan reprodusere feilen

Trinn for å gjenskape:

  1. Installer PP365
  2. Opprett et kommunikasjonsområde
  3. Åpne console og Sources
  4. Søk etter e.g. "IdeaRegistrationCommand"
  5. Sett gjerne breakpoint i fila, observer at den laster

Forventet oppførsel

Løsningene bør ikke deployes globalt. For denne konkrete pakka så kan vi droppe skipFeatureDeployment, og heller legge til appen på porteføljeområdet der den skal brukes som en del av installasjon/template.

Uansett er det så niche å bruke denne funksjonaliteten per nå, at jeg vil heller at vi må legge den inn manuelt enn at vi deployer globalt.

Skjermbilder

No response

Ytterligere informasjon

No response

Hvilke nettleser(e) oppleves feilen på?

Microsoft Edge

Versjon

1.8.x (siste stabile)

Relevant logg

No response

@tarjeieo tarjeieo added bug Something isn't working priority: high Issue with high priority complexity: medium Issue/PR with medium complexity labels Sep 12, 2023
@tarjeieo tarjeieo added this to the 1.9 milestone Sep 12, 2023
@Remi749
Copy link
Collaborator

Remi749 commented Sep 13, 2023

@tarjeieo, tenker du at vi skal legge til et parameter i install-scriptet om man ønsker å installere pakka på porteføljen? Eller at brukere selv legger til appen på porteføljen sin manuelt i etterkant?

@tarjeieo
Copy link
Member Author

Hadde egentlig tenkte å installere den alltid på porteføljeområdet... Det sagt så kanskje vi bare skal la folk legge den til manuelt. Den er jo overdue for en oppgradering - og det har jo skjedd litt både her og der rundt dette temaet!

@tarjeieo tarjeieo added priority: medium Issue with medium priority and removed priority: high Issue with high priority labels Sep 15, 2023
@olemp olemp mentioned this issue Sep 27, 2023
37 tasks
@Remi749 Remi749 assigned olemp and unassigned Remi749 Oct 25, 2023
@olemp olemp removed their assignment Oct 25, 2023
@Remi749 Remi749 added priority: high Issue with high priority and removed priority: medium Issue with medium priority labels Nov 6, 2023
@olemp
Copy link
Collaborator

olemp commented Nov 23, 2023

@Remi749 @tarjeieo Hva er status her? Jeg er litt usikker på hvordan det hele henger sammen. Skal det være custom actions på lister på både porteføljeområdet og prosjektnivå? For porteføljenivå er det jo rett frem å legge inn custom actions i Porteføljemalen. For prosjekter blir det nok å oppdatere site designet eller legge det i prosjektmalen (men usikker på om klientside provisjoneringen støtter dette per nå).

@tarjeieo
Copy link
Member Author

Trenger vel egentlig bare på porteføljeområdet? Så om vi scoper til porteføljen så tror jeg vi er good for nå

@Remi749
Copy link
Collaborator

Remi749 commented Nov 23, 2023

I PortfolioExtensions har vi Idébehandling opplegget og Footer. Footer må vises på Portefølje og Prosjekt, Idébehandling er eksklusiv Porteføljenivå

@olemp olemp unassigned olemp Dec 14, 2023
@olemp olemp assigned Remi749 and unassigned olemp Dec 21, 2023
@Remi749
Copy link
Collaborator

Remi749 commented Jan 5, 2024

... For porteføljenivå er det jo rett frem å legge inn custom actions i Porteføljemalen. ...

Dersom dette er det som må gjøres er dette allerede utført. Dette ble gjort her: 8aae362

@Remi749 Remi749 closed this as completed Jan 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working complexity: medium Issue/PR with medium complexity priority: high Issue with high priority
Projects
None yet
Development

No branches or pull requests

3 participants