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

[Briefing] Ajout des images de la carte au pdf #1954

Open
wants to merge 12 commits into
base: main
Choose a base branch
from

Conversation

maximeperraultdev
Copy link
Collaborator

Related Pull Requests & Issues


  • Tests E2E (Cypress)

@maximeperraultdev maximeperraultdev force-pushed the maxime/brief/add_map_to_pdf branch 5 times, most recently from 8a9cec4 to 3778445 Compare December 23, 2024 16:42
@maximeperraultdev
Copy link
Collaborator Author

@claire2212 Tu peux relire d'un point de vue technique pour que ça aille plus vite. ll faut qu'on fasse quelques aller retours ux/ui car j'ai quelques questions que j'ai mis dans le chan mattermost

@maximeperraultdev maximeperraultdev force-pushed the maxime/brief/add_map_to_pdf branch from 3778445 to 659faa1 Compare December 26, 2024 10:50
@maximeperraultdev maximeperraultdev force-pushed the maxime/brief/add_map_to_pdf branch from da5ad7f to 8ec8588 Compare January 6, 2025 08:49
renderOrder: (a, b) => b.get('area') - a.get('area'),
source: layersVectorSourceRef.current,
style: feature => getDashboardStyle(feature),
zIndex: Layers.DASHBOARD.zIndex
Copy link
Collaborator

Choose a reason for hiding this comment

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

Layers.EXPORT_PDF.zIndex plutôt?

)
}

const StyledLinkButton = styled(Button)<{ $isDisabled: boolean }>`
const StyledLinkButton = styled(Button)<{ disabled: boolean }>`
Copy link
Collaborator

Choose a reason for hiding this comment

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

il faut préfixer les props de style par un $ (demandé par styled-component)

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Il me semble que c'est pour les props qui ne doivent pas se retrouver dans le DOM. Et la props disabled est autorisé :)

@claire2212 claire2212 force-pushed the maxime/brief/add_map_to_pdf branch from 957310c to 33a4b62 Compare January 6, 2025 14:08
maximeperrault and others added 10 commits January 8, 2025 14:33
Bumps the non-major-dev-dependencies group in /frontend with 9 updates:

| Package | From | To |
| --- | --- | --- |
| [@import-meta-env/cli](https://github.com/runtime-env/import-meta-env/tree/HEAD/packages/cli) | `0.7.1` | `0.7.2` |
| [@import-meta-env/unplugin](https://github.com/runtime-env/import-meta-env/tree/HEAD/packages/unplugin) | `0.6.0` | `0.6.2` |
| [@swc/core](https://github.com/swc-project/swc) | `1.9.3` | `1.10.4` |
| [cypress](https://github.com/cypress-io/cypress) | `13.16.0` | `13.17.0` |
| [dotenv](https://github.com/motdotla/dotenv) | `16.4.5` | `16.4.7` |
| [eslint-plugin-react](https://github.com/jsx-eslint/eslint-plugin-react) | `7.37.2` | `7.37.3` |
| [lint-staged](https://github.com/lint-staged/lint-staged) | `15.2.10` | `15.3.0` |
| [type-fest](https://github.com/sindresorhus/type-fest) | `4.29.0` | `4.31.0` |
| [vite-tsconfig-paths](https://github.com/aleclarson/vite-tsconfig-paths) | `5.1.3` | `5.1.4` |

Updates `@import-meta-env/cli` from 0.7.1 to 0.7.2
- [Release notes](https://github.com/runtime-env/import-meta-env/releases)
- [Changelog](https://github.com/runtime-env/import-meta-env/blob/main/packages/cli/CHANGELOG.md)
- [Commits](https://github.com/runtime-env/import-meta-env/commits/cli0.7.2/packages/cli)

Updates `@import-meta-env/unplugin` from 0.6.0 to 0.6.2
- [Release notes](https://github.com/runtime-env/import-meta-env/releases)
- [Changelog](https://github.com/runtime-env/import-meta-env/blob/main/packages/unplugin/CHANGELOG.md)
- [Commits](https://github.com/runtime-env/import-meta-env/commits/cli0.6.2/packages/unplugin)

Updates `@swc/core` from 1.9.3 to 1.10.4
- [Release notes](https://github.com/swc-project/swc/releases)
- [Changelog](https://github.com/swc-project/swc/blob/main/CHANGELOG.md)
- [Commits](swc-project/swc@v1.9.3...v1.10.4)

Updates `cypress` from 13.16.0 to 13.17.0
- [Release notes](https://github.com/cypress-io/cypress/releases)
- [Changelog](https://github.com/cypress-io/cypress/blob/develop/CHANGELOG.md)
- [Commits](cypress-io/cypress@v13.16.0...v13.17.0)

Updates `dotenv` from 16.4.5 to 16.4.7
- [Changelog](https://github.com/motdotla/dotenv/blob/master/CHANGELOG.md)
- [Commits](motdotla/dotenv@v16.4.5...v16.4.7)

Updates `eslint-plugin-react` from 7.37.2 to 7.37.3
- [Release notes](https://github.com/jsx-eslint/eslint-plugin-react/releases)
- [Changelog](https://github.com/jsx-eslint/eslint-plugin-react/blob/master/CHANGELOG.md)
- [Commits](jsx-eslint/eslint-plugin-react@v7.37.2...v7.37.3)

Updates `lint-staged` from 15.2.10 to 15.3.0
- [Release notes](https://github.com/lint-staged/lint-staged/releases)
- [Changelog](https://github.com/lint-staged/lint-staged/blob/master/CHANGELOG.md)
- [Commits](lint-staged/lint-staged@v15.2.10...v15.3.0)

Updates `type-fest` from 4.29.0 to 4.31.0
- [Release notes](https://github.com/sindresorhus/type-fest/releases)
- [Commits](sindresorhus/type-fest@v4.29.0...v4.31.0)

Updates `vite-tsconfig-paths` from 5.1.3 to 5.1.4
- [Release notes](https://github.com/aleclarson/vite-tsconfig-paths/releases)
- [Commits](aleclarson/vite-tsconfig-paths@v5.1.3...v5.1.4)

---
updated-dependencies:
- dependency-name: "@import-meta-env/cli"
  dependency-type: direct:development
  update-type: version-update:semver-patch
  dependency-group: non-major-dev-dependencies
- dependency-name: "@import-meta-env/unplugin"
  dependency-type: direct:development
  update-type: version-update:semver-patch
  dependency-group: non-major-dev-dependencies
- dependency-name: "@swc/core"
  dependency-type: direct:development
  update-type: version-update:semver-minor
  dependency-group: non-major-dev-dependencies
- dependency-name: cypress
  dependency-type: direct:development
  update-type: version-update:semver-minor
  dependency-group: non-major-dev-dependencies
- dependency-name: dotenv
  dependency-type: direct:development
  update-type: version-update:semver-patch
  dependency-group: non-major-dev-dependencies
- dependency-name: eslint-plugin-react
  dependency-type: direct:development
  update-type: version-update:semver-patch
  dependency-group: non-major-dev-dependencies
- dependency-name: lint-staged
  dependency-type: direct:development
  update-type: version-update:semver-minor
  dependency-group: non-major-dev-dependencies
- dependency-name: type-fest
  dependency-type: direct:development
  update-type: version-update:semver-minor
  dependency-group: non-major-dev-dependencies
- dependency-name: vite-tsconfig-paths
  dependency-type: direct:development
  update-type: version-update:semver-patch
  dependency-group: non-major-dev-dependencies
...

Signed-off-by: dependabot[bot] <[email protected]>
@maximeperraultdev maximeperraultdev force-pushed the maxime/brief/add_map_to_pdf branch from 886b598 to 7d06623 Compare January 8, 2025 13:36
@maximeperraultdev maximeperraultdev force-pushed the maxime/brief/add_map_to_pdf branch from 7d06623 to 89d5fb5 Compare January 8, 2025 13:45
@maximeperraultdev maximeperraultdev force-pushed the maxime/brief/add_map_to_pdf branch from 46da7ca to 90c6127 Compare January 9, 2025 11:09
Copy link

sonarqubecloud bot commented Jan 9, 2025

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.

[Tableau de bord & Brief] Générer un pdf avec la carto
3 participants