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

[DONE] Feature add personal meeting room #1073

Conversation

LoicBonavent
Copy link
Collaborator

New functionality to manage a personal meeting room in the meetings module.

  • Addition of is_personal field in base
  • Creation of a personal meeting room, for each user, on 1st connection to the meetings module.
  • This room always appears in 1st position
  • This room cannot be deleted
  • This room is permanent; there is no date management (neither when modifying, nor when sending an invitation email).

Minor bug fixes in the meetings module (voice bridge no., rights, etc.)

@LoicBonavent LoicBonavent self-assigned this Mar 8, 2024
@LoicBonavent
Copy link
Collaborator Author

Gère l'Issue #1049

pod/locale/fr/LC_MESSAGES/django.po Outdated Show resolved Hide resolved
pod/locale/fr/LC_MESSAGES/django.po Outdated Show resolved Hide resolved
pod/meeting/templates/meeting/add_or_edit.html Outdated Show resolved Hide resolved
pod/meeting/views.py Show resolved Hide resolved
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.

Merci pour les corrections ;)

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.

Hello, merci pour cette contribution. C'est dommage qu'on est pas vu ensemble avant pour cela. Je pense en effet qu'il y a bcp plus simple sans modification de base de données et de formulaire. En effet, on pourrait créer en base une "room" aynt comme nom le username de l'utilisateur prefixée de "personnal" ce qui permettrait de savoir si c'est une reunion virtuelle personnelle. plutot que de la faire apparaitre au même endroit que les autres, deux boutons : "démarrer/rejoindre" et copier le lien d'accès direct permettrait de gerer cette reunion personnelle. Qu'en penses-tu ?

@LoicBonavent
Copy link
Collaborator Author

LoicBonavent commented Mar 12, 2024

Suite à notre point avec Nicolas, j'ai réalisé les modifications suivantes :

  • Le nom de la réunion est en lecture seule pour une salle de réunion personnelle.
  • Un propriétaire additionnel ne peut modifier une salle de réunion personnelle d'un autre usager.
  • Ajout d'un liseré bleu pour les salles de réunion personnelles.
  • En tant que propriétaire additionnel, les réunions actives des autres usagers sont vus également comme actives chez soi.

pod/meeting/forms.py Show resolved Hide resolved
pod/meeting/models.py Show resolved Hide resolved
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, des petits details mais ca le fait pour une première version. merci

pod/meeting/forms.py Show resolved Hide resolved
@ptitloup ptitloup merged commit b931a94 into EsupPortail:develop Mar 14, 2024
3 checks passed
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.

None yet

3 participants