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.

Sign up for GitHub

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

Jump to bottom

🚧(dashboard) add consent models #235

Merged
merged 1 commit into from
Nov 22, 2024
Merged

🚧(dashboard) add consent models #235

merged 1 commit into from
Nov 22, 2024

Conversation

ssorin
Copy link
Collaborator

@ssorin ssorin commented Nov 19, 2024

Added

  • add consent models
  • create core app
  • add basic consent admin
  • create entity and delivery_point models
  • add basic entity and delivery_point admin

jmaupetit
jmaupetit reviewed Nov 20, 2024
View reviewed changes
Copy link
Member

@jmaupetit jmaupetit left a comment

Choose a reason for hiding this comment

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

Oh wow, we are on a good path 😻

src/dashboard/Pipfile Outdated Show resolved Hide resolved
src/dashboard/apps/consent/migrations/0002_remove_chargepointconsent_charge_point_and_more.py Outdated Show resolved Hide resolved
src/dashboard/apps/core/models/abstract.py Outdated Show resolved Hide resolved
src/dashboard/apps/core/models/abstract.py Outdated Show resolved Hide resolved
src/dashboard/apps/core/models/abstract.py Outdated Show resolved Hide resolved
src/dashboard/apps/core/models/entity.py Outdated Show resolved Hide resolved
src/dashboard/apps/core/models/delivery_point.py Outdated Show resolved Hide resolved
src/dashboard/apps/consent/models.py Outdated Show resolved Hide resolved
src/dashboard/apps/consent/models.py Outdated Show resolved Hide resolved
src/dashboard/apps/consent/models.py Outdated Show resolved Hide resolved
@ssorin ssorin force-pushed the feature/dashboard/add-consent-models branch from b1cd4ea to d65eed1 Compare November 21, 2024 09:01
@ssorin ssorin added the feature label Nov 21, 2024
@ssorin ssorin self-assigned this Nov 21, 2024
@ssorin ssorin added dashboard Item related to the Dashboard service WIP labels Nov 21, 2024
@ssorin ssorin force-pushed the feature/dashboard/add-consent-models branch 4 times, most recently from a40f224 to ef1efcf Compare November 21, 2024 15:43
@ssorin ssorin force-pushed the feature/dashboard/add-consent-models branch from ef1efcf to da71533 Compare November 21, 2024 15:51
jmaupetit
jmaupetit reviewed Nov 21, 2024
View reviewed changes
src/dashboard/apps/core/base.py Outdated Show resolved Hide resolved
src/dashboard/apps/core/models.py Show resolved Hide resolved
src/dashboard/apps/core/tests/test_models.py Outdated Show resolved Hide resolved
@ssorin ssorin force-pushed the feature/dashboard/add-consent-models branch from 7516200 to 2e54f81 Compare November 22, 2024 08:35
jmaupetit
jmaupetit approved these changes Nov 22, 2024
View reviewed changes
Copy link
Member

@jmaupetit jmaupetit left a comment

Choose a reason for hiding this comment

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

Ship it!

@jmaupetit jmaupetit marked this pull request as ready for review November 22, 2024 08:42
@ssorin
✨(dashboard) add consent model
0e2edb6 
add consent models
add basic content admin
create core app
add entity and delivery_point models
add basic entity and delivery_point admin
add unitest for models consent and core
 update CHANGELOG.md
@ssorin ssorin force-pushed the feature/dashboard/add-consent-models branch from 2e54f81 to 0e2edb6 Compare November 22, 2024 08:55
@ssorin ssorin merged commit 773e47b into main Nov 22, 2024
21 checks passed
@ssorin ssorin deleted the feature/dashboard/add-consent-models branch November 22, 2024 10:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jmaupetit jmaupetit jmaupetit approved these changes

Assignees

@ssorin ssorin

Labels
dashboard Item related to the Dashboard service feature
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ssorin @jmaupetit