Skip to content

✨ introduce specialists #336

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
Merged
merged 196 commits into from
May 3, 2023
Merged

✨ introduce specialists #336

merged 196 commits into from
May 3, 2023

Conversation

@thibaudbrault
Copy link
Contributor

@thibaudbrault thibaudbrault commented Apr 20, 2023 •
edited
Loading

This introduces the notion of some users being specialists in some subjects (aka tag labels). Specialists will have more permissions on questions and answers which are tagged with their specialties, though no permissions are introduced here. The specialties of a user are displayed on their profile. This does not introduce means to define specialties through the UI, this will come later.

See also #334.

thibaudbrault and others added 30 commits March 14, 2023 17:22
@thibaudbrault
🧪 add playwright
6f3a10b
✅ test can bypass authentification check
263b0c7
✅ test use fake user
5a7ded4
✅ use fake information for code
6f00752
🐛 fix bug question not sent during test
8edc720
@thibaudbrault
✅ create multiple tests to check that the site is still working
15a2415
✅ correct some tests + create token when laucnhing tests
e8b3e6b
@thibaudbrault
✅ tests use token and empty database before running
9b46c27
🧪 add algolia to tests + modif folder tructure for tests
2bd3eac
✏️ remove   in tips that were displayed on screen
15c4043
@thibaudbrault
✅ tests with algolia work - text with tags do not
2f31f62
@thibaudbrault
✅ test pass except last one
d5c51ae
@thibaudbrault
✅ tests with tag filter work
11ffb31
✅ create user and fetch tags before running tests
c0212eb
@thibaudbrault
👷 added test job in circleci config
935c6bb
@thibaudbrault
💚 add install packages directory in test job
9d2a0d0
@thibaudbrault
💚 fix typo in circleci config
137eecf
@thibaudbrault
💚 fix problem with app-dir for test job
999c0cc
@thibaudbrault
✅ fix mistake in test 9
e75953b
@thibaudbrault
📝 add docs page for testing
9494b54
@thibaudbrault
💚 add command in test job
d8154e9
@thibaudbrault
💚 add environment variables for new_service command
eea2b9a
@thibaudbrault
🗃️ modification datamodel to fix circleci errors
a8af999
@thibaudbrault
💚 add step to check prisma version
2f77327
✏️ fix typos in test page of the docs
dc96264
@thibaudbrault
👷 migrate prisma server from 1.20 to 1.34.10
03d0edc
@thibaudbrault
💚 add environment variables fro test job
646591f
@thibaudbrault
💚 add wait-on for command run new_service
68d5fca
@thibaudbrault
💚 make run new_service run in the background
358d03b
@thibaudbrault
💚 add env variables to test command
d620eac
@thibaudbrault thibaudbrault changed the base branch from main to testPlaywright April 26, 2023 08:22
hgwood
hgwood reviewed Apr 26, 2023
View reviewed changes
hgwood
hgwood reviewed May 3, 2023
View reviewed changes
@hgwood hgwood changed the title User specialities ✨ introduce specialists May 3, 2023
@hgwood hgwood marked this pull request as ready for review May 3, 2023 12:07
hgwood
hgwood reviewed May 3, 2023
View reviewed changes
Copy link
Contributor

@hgwood hgwood left a comment

Choose a reason for hiding this comment

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

Je viens de me rendre compte que "spécialités" en Anglais c'est "specialties" et pas "specialities". Il faut renommer les fichiers et variables pour corriger.

@thibaudbrault thibaudbrault changed the base branch from testPlaywright to main May 3, 2023 13:41
@thibaudbrault thibaudbrault merged commit 7f04f6d into main May 3, 2023
@thibaudbrault thibaudbrault deleted the userSpecialities branch June 8, 2023 13:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@hgwood hgwood hgwood left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@thibaudbrault @hgwood