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

Adding officerships to web-auth #27

Merged
merged 88 commits into from
Aug 20, 2024
Merged

Adding officerships to web-auth #27

merged 88 commits into from
Aug 20, 2024

Conversation

COMTOP1
Copy link
Member

@COMTOP1 COMTOP1 commented Jun 24, 2024 •
edited
Loading

Adding the ability to add, edit and delete all officership functions

COMTOP1 added 16 commits June 24, 2024 06:30
@COMTOP1 COMTOP1 linked an issue Jun 26, 2024 that may be closed by this pull request
Having the ability to modify officerships would be good #28
Closed
COMTOP1 added 24 commits August 14, 2024 00:28
@COMTOP1
Moved remaining officership db sql statements to squirrel
85a8317
@COMTOP1
Moved api db sql statements to squirrel
2435d39
@COMTOP1
Fixing linting
bb52a9c
@COMTOP1
Fixing queries to actually be postgres and use args
2e318c1
@COMTOP1
Minor styling change
ec52a52
@COMTOP1
Changing uni username to string as it can't be null
945865c
@COMTOP1
Removing all field data
a3484b5
@COMTOP1
Fixing getOfficershipTeamMember to actually get the single entity
7567aaf
@COMTOP1
Renaming the variable
39411e3
@COMTOP1
Updating
1cc5c0c
@COMTOP1
Fixing to return the correct number of variables
d2e26af
@COMTOP1
Removed unused function
052f014
@COMTOP1
Adding a function to get the officerships not in team
69b2900
@COMTOP1
Fixing incorrect returning values
d05f07a
@COMTOP1
Minor change to show "was" if the officership is not current
cd9f094
@COMTOP1
Fixing incorrect parameter
1df2804
@COMTOP1
Adding officerships not in team to the team page
868b602
@COMTOP1
Adding functions to add and remove officerships from teams
70557b6
@COMTOP1
Adding front end ability to add and remove officerships from teams
04cc2fa
@COMTOP1
Fixing linting
6706b1b
@COMTOP1
Fixing for field
3e0318a
@COMTOP1
Fixing returning values
b7ba1f0
@COMTOP1
Moving to use the better search dropdowns
ad1ae0a
@COMTOP1
Adding username to search fields
606ccaa
@COMTOP1 COMTOP1 marked this pull request as ready for review August 20, 2024 21:21
@COMTOP1 COMTOP1 merged commit b4050ff into main Aug 20, 2024
8 checks passed
@COMTOP1 COMTOP1 deleted the liam.burnand/adding-officerships branch August 20, 2024 21:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Having the ability to modify officerships would be good
1 participant
@COMTOP1