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

Unpublish articles when they are not used in taxonomy or learningpaths #1691

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

Jonas-C
Copy link
Contributor

@Jonas-C Jonas-C commented Feb 15, 2023

Fixes https://github.com/NDLANO/Issues/issues/3398
Tror det skal funke, men har ikke fått testet det ordentlig ennå på grunn av feil i indeksering og draft-api.

@Jonas-C Jonas-C requested a review from a team February 15, 2023 13:01
@cypress
Copy link

cypress bot commented Feb 15, 2023

Passing run #11229 ↗︎

0 52 0 0 Flakiness 0

Details:

isLoading -> isInitialLoading to avoid inifite load
Project: editorial-frontend Commit: ea307da6c7
Status: Passed Duration: 11:16 💡
Started: Mar 22, 2023 2:46 PM Ended: Mar 22, 2023 2:57 PM

This comment has been generated by cypress-bot as a result of this project's GitHub integration settings.

@cypress
Copy link

cypress bot commented Feb 15, 2023

Passing run #11230 ↗︎

0 52 0 0 Flakiness 0

Details:

Merge ea307da into 1427a7d...
Project: editorial-frontend Commit: d62e9251aa ℹ️
Status: Passed Duration: 11:11 💡
Started: Mar 22, 2023 2:46 PM Ended: Mar 22, 2023 2:57 PM

This comment has been generated by cypress-bot as a result of this project's GitHub integration settings.

@Jonas-C Jonas-C requested a review from gunnarvelle March 7, 2023 12:47
@gunnarvelle
Copy link
Member

Testa å legge ny ressurs til to emner og fjerna dei fra emnet etterpå. Skjedde ingenting med artikkelen.

@Jonas-C
Copy link
Contributor Author

Jonas-C commented Mar 22, 2023

Jeg tror det er noe rart i backend. Vi får en valideringsfeil fra backend om at artikkelen vi prøver å unpublishe bruker i læringsstier, selv om den er nyopprettet. Noen som har peiling?

@gunnarvelle
Copy link
Member

Trudde vi hadde fiksa akkurat det.

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.

2 participants