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

feat: ❇️ add tutorial to secure a firearm #159

Merged
merged 81 commits into from
Oct 10, 2023

Conversation

NadeigeC
Copy link
Contributor

Voir les maquettes FIGMA

@NadeigeC NadeigeC added the feature New feature / functionality label May 23, 2023
@NadeigeC NadeigeC self-assigned this May 23, 2023
@NadeigeC NadeigeC changed the title [WIP] feat: ❇️ : Add tutorial to secure a firearm [WIP] feat: ❇️ : add tutorial to secure a firearm May 23, 2023
@NadeigeC NadeigeC force-pushed the feat/add-tutorial-to-secure-a-firearm branch 2 times, most recently from c81486d to 98c4f47 Compare May 23, 2023 12:45
@NadeigeC NadeigeC force-pushed the feat/add-tutorial-to-secure-a-firearm branch from 98c4f47 to 47d5fde Compare May 23, 2023 12:52
@NadeigeC NadeigeC changed the title [WIP] feat: ❇️ : add tutorial to secure a firearm [WIP] feat: ❇️ add tutorial to secure a firearm May 30, 2023
@NadeigeC NadeigeC requested review from laruiss and removed request for laruiss June 1, 2023 20:39
@NadeigeC NadeigeC marked this pull request as draft June 7, 2023 12:37
@NadeigeC NadeigeC force-pushed the feat/add-tutorial-to-secure-a-firearm branch from 4735cf9 to d1614b0 Compare September 28, 2023 13:14
@NadeigeC NadeigeC force-pushed the feat/add-tutorial-to-secure-a-firearm branch from cde75a0 to d466be9 Compare September 28, 2023 15:09
@NadeigeC NadeigeC marked this pull request as ready for review September 29, 2023 11:23
@NadeigeC NadeigeC changed the title [WIP] feat: ❇️ add tutorial to secure a firearm feat: ❇️ add tutorial to secure a firearm Sep 29, 2023
* chore: 🔧 add start script

* refactor: 🔥 remove unused code

* chore: 🚨 improve eslint config

* test: ✅ add new test to identificate a firearm

* test: ✅ add new test to securing firearm

* fix: ✅ fix errors and variable name

* fix: ✅ triying to fix video loading issue

* test: ✅ add new test to identificate firearm fiability

* chore: ⬆️ upgrade dependencies in frontend

* test: ✅ add new test to identificate revolver typology

* test: ✅ add new test to identificate revolver typology

* test: ✅ add new test to securing shoulder bolt rifle

* test: ✅ add new test to securing old mechanism pistol

* test: 🔨 add new commands for tests e2e

* test: ✅ update existing tests with new commands

* test: ✅ add new test for check if it's FSI or Civilians recommendations

* test: ✅ update tests and files

* test: ✅ update tests and files

---------

Co-authored-by: Nadeige COUTHON <[email protected]>
@nutfdt nutfdt self-assigned this Oct 6, 2023
* feat: ✨ handle video in options steps

* style: 💄 updated revolver securing first steps video

* style: 💄 updated revolver securing first steps video

* style: 💄 update revolver barrel video and instructions

* style: 💄 update revolver choice display
@nutfdt nutfdt merged commit e1edd0a into develop Oct 10, 2023
6 checks passed
@nutfdt nutfdt deleted the feat/add-tutorial-to-secure-a-firearm branch October 10, 2023 11:30
nutfdt added a commit that referenced this pull request Oct 10, 2023
thomashbrnrd pushed a commit that referenced this pull request Oct 24, 2023
* refactor: ♻️ remove swiper component and add page instead

* refactor: ♻️ remove swiper component and add page instead

* refactor: ♻️ refactoring StartPage

* chore: ⬆️ Upgrade dependancies

* feat: 🐛 Fix routing

* feat: ✨ Add 2 buttons on StartPage

* feat: ✨ Add new stepper to securing firearm

* style: 🎨

* style: 💄 Move stepper title inline

* feat: 💄 Update Basegun baseline

* feat: ✨ Add tutorial content to secure a firearm

* feat: 🎨 Add new identification guide

* WIP identification stepper

* fix: 🐛 Fix issues in routing

* chore: ⬆️ Upgrade dependancies

* refactor: 🎨 Replace OnboardingPage by Swiper

* refactor: 🎨

* refactor: ♻️ Refactoring on page result

* fix: 🎨 Improve navigation in identificationGuide

* style: 💄 Add style on ResultPage

* feat: ⚡ WIP : Switch to different routes for each tutorial

* add test video to secure a firearm

* feat: ✨ Add securing-firearm-util.js file

* fix: ✅ Fix tests E2E in securing branch

* split securing-firearms-utils.js in 12 specifics files

* feat: ⚡ Fix selectAmmo for dummy typologies

* WIP

* WIP MES

* upgrade dependencies

* Wip MES

* Add new photos and videos in several typologies

* Add specific select options for revolver

* Change legal category if revolver black powder

* Add step 3 options for revolver

* Add content and style on accordions title

* wip css integration

* add minified header on mobile

* style: 💄 Add whole logo only on start pages

* style: 💄 Add whole logo only on start pages

* fix DsfrRadioButton rich image ratio

* add accordion design to revolver-1873-fr

* add video parts to accordion design

* refactor: ♻️ Remove useless code and clean code

* refactor: ♻️ Renaming components

* refactor: ♻️ Add highlight bold on tutorial texts manipulations

* refactor: ♻️ remove useless const identificationTutorial

* wip-refacto

* refactor: ♻️ improve navigation in guideSecuringFirearms

* perf: ♻️ Downsize too big photos

* refactoring titles styles

* chore: ⬇️ Downgrade Cypress from 13.1.0 to 12.17.4

* refactor: ⚡ Make accordion tutorial only visible for revolver 1873 typology

* style: 💄 add new routes names and new icons on securing tutorial

* feat: ✨ add a different security message for civilians

* fix: 🐛 fix tutorial display for revolver different from 1892 and 1873fr

* style: 💄 add new images with focus for securing firearm

* style: 💄 add new photos with circle

* refactoring titles styles

* refactor: ♻️ wip refactor selectOption component

* fix: 🐛 fix navigation for revolver typology

* refactor: ♻️ Fix refactoring selectOption component

* chore: ⬆️ upgrade dependencies

* [WIP] test: 🧪 tests/e2e (#176)

* chore: 🔧 add start script

* refactor: 🔥 remove unused code

* chore: 🚨 improve eslint config

* test: ✅ add new test to identificate a firearm

* test: ✅ add new test to securing firearm

* fix: ✅ fix errors and variable name

* fix: ✅ triying to fix video loading issue

* test: ✅ add new test to identificate firearm fiability

* chore: ⬆️ upgrade dependencies in frontend

* test: ✅ add new test to identificate revolver typology

* test: ✅ add new test to identificate revolver typology

* test: ✅ add new test to securing shoulder bolt rifle

* test: ✅ add new test to securing old mechanism pistol

* test: 🔨 add new commands for tests e2e

* test: ✅ update existing tests with new commands

* test: ✅ add new test for check if it's FSI or Civilians recommendations

* test: ✅ update tests and files

* test: ✅ update tests and files

---------

Co-authored-by: Nadeige COUTHON <[email protected]>

* feat: ✨ handle video in options steps (#187)

* feat: ✨ handle video in options steps

* style: 💄 updated revolver securing first steps video

* style: 💄 updated revolver securing first steps video

* style: 💄 update revolver barrel video and instructions

* style: 💄 update revolver choice display

* style: 💄 updated shotgun videos

* test: ✅ skip fiability test

---------

Co-authored-by: David Mendy <[email protected]>
Co-authored-by: nutfdt <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature / functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants