Skip to content

[DONE] Fix wrong url of paginated sub-themes #789

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

Conversation

pampletousse
Copy link
Contributor

Fix wrong url of paginated sub-themes on page2 have /channel/theme/subtheme url which lead to 404 -> fix to /channel/subtheme

…age2 have /channel/theme/subtheme url which lead to 404 -> fix to /channel/subtheme
Copy link
Contributor

@ptitloup ptitloup left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok pour moi

@ptitloup
Copy link
Contributor

suite aux remarques de @Badatos je me dis egalement qu'il faudrait passer par des données sous forme de dataset (data-url par exemple ou data-slug) plutot que par variables

Copy link
Collaborator

@Badatos Badatos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ca me semble pas mal. merci pour les changements ;)

Copy link
Contributor

@ptitloup ptitloup left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Top ! merci

@ptitloup ptitloup merged commit cd4d98d into EsupPortail:develop Mar 24, 2023
ptitloup added a commit that referenced this pull request Apr 12, 2023
* 🐛 podfile fix pagination display (#787)
* improve clean_video_files and code cleaning (#782)
* Fix wrong url of paginated sub-themes (#789)
* Add "help" command to makefile (#792)
* Ptitloup/fix thumbnail (#790)
* Hide channel desc (#794)
* fix podfile (#798)
* Correct podfile sidebar (#799)
* Ptitloup/fix link playlist (#801)
* Fix video theme (#800)


---------
Co-authored-by: MattBild <34771705+mattbild@users.noreply.github.com>
Co-authored-by: Olivier Bado-Faustin <bado@unice.fr>
Co-authored-by: pampletousse <51699553+pampletousse@users.noreply.github.com>
vsabatie pushed a commit to vsabatie/Pod that referenced this pull request Apr 20, 2023
* 🐛 podfile fix pagination display (EsupPortail#787)
* improve clean_video_files and code cleaning (EsupPortail#782)
* Fix wrong url of paginated sub-themes (EsupPortail#789)
* Add "help" command to makefile (EsupPortail#792)
* Ptitloup/fix thumbnail (EsupPortail#790)
* Hide channel desc (EsupPortail#794)
* fix podfile (EsupPortail#798)
* Correct podfile sidebar (EsupPortail#799)
* Ptitloup/fix link playlist (EsupPortail#801)
* Fix video theme (EsupPortail#800)

---------
Co-authored-by: MattBild <34771705+mattbild@users.noreply.github.com>
Co-authored-by: Olivier Bado-Faustin <bado@unice.fr>
Co-authored-by: pampletousse <51699553+pampletousse@users.noreply.github.com>
@pampletousse pampletousse deleted the pampletousse/fix-url_paginated_sub_themes branch January 24, 2024 15:25
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.

3 participants