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 bookmarks for objects #781

Merged
merged 2 commits into from
Oct 10, 2024
Merged

feat: add bookmarks for objects #781

merged 2 commits into from
Oct 10, 2024

Conversation

vjousse
Copy link
Collaborator

@vjousse vjousse commented Oct 1, 2024

🔧 Problem

The first version of the object interface doesn't manage bookmarks.

🍰 Solution

Implement the bookmark mechanism already available for the food and textile sections.

🚨 Points to watch/comments

The toString on the Query object to find a default name for the bookmark may be questionnable.

🏝️ How to test

Go on the review app https://ecobalyse-pr781.osc-fr1.scalingo.io/#/object/simulator and try to save/load simulations for the object interface using the bottom right tab.

Screenshot 2024-10-01 at 14-41-57 Simulateur Ecobalyse

@vjousse vjousse marked this pull request as ready for review October 2, 2024 08:03
@vjousse vjousse requested a review from n1k0 October 7, 2024 07:38
Copy link
Member

@n1k0 n1k0 left a comment

Choose a reason for hiding this comment

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

Looks promising! We should have the example name in the bookmark name text input when one is selected, as in textile and food though.

src/Views/Comparator.elm Outdated Show resolved Hide resolved
src/Page/Object.elm Outdated Show resolved Hide resolved
@n1k0 n1k0 self-requested a review October 10, 2024 13:05
Copy link
Member

@n1k0 n1k0 left a comment

Choose a reason for hiding this comment

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

Let's land this so we can iterate further.

@vjousse vjousse merged commit 8c58441 into master Oct 10, 2024
6 checks passed
@vjousse vjousse deleted the feat/add-object-bookmarks branch October 10, 2024 13:10
vjousse pushed a commit that referenced this pull request Nov 7, 2024
## [2.5.0](https://github.com/MTES-MCT/ecobalyse/compare/v2.4.0..v2.5.0)
(2024-11-06)



### 🚀 Features

- Add bookmarks for objects
([#781](#781))
- Add object explorer pages.
([#803](#803))
- Distinguish Objects from Veli.
([#813](#813))
- Display score without durability
([#815](#815))
- Textile export
([#808](#808))
- Object export
([#812](#812))

### 🪲 Bug Fixes

- Create object encrypted file for versions
([#800](#800))
- Improve object simulator.
([#799](#799))
- Fix encoded display name field.
([#820](#820))

### 🚜 Refactor

- Aggregate in python
([#794](#794))
- Turn food process category into a list
([#795](#795))
- Aggregate in python
([#807](#807))

### ⚙️ Miscellaneous Tasks

- Upgrade dependencies to their latest version, Oct. 2024.
([#801](#801))
- Add tolerance to tests comparison
([#810](#810))
- *(data)* New export
([#819](#819))

### ◀️ Revert

- "refactor: aggregate in python"
([#806](#806))

<!-- generated by git-cliff -->

Signed-off-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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