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

Add linked account creation from the Discord bot #301

Merged
merged 18 commits into from
Aug 10, 2024
Merged

Add linked account creation from the Discord bot #301

merged 18 commits into from
Aug 10, 2024

Commits on Jul 25, 2024

  1. Create migration for account invitation with Discord link

    @daanbreur
    daanbreur committed Jul 25, 2024
    Configuration menu
    Copy the full SHA
    96f5217 View commit details
    Browse the repository at this point in the history

  2. Removes the unique constraint from discord_id in `ctfnote_private.inv… 

    …itation_link`.

It is preferred we unique this column however whenever you accidentally dismiss the discord bot and want to make a new token you cant. This is why we removed the unique constraint for now.
    @daanbreur
    daanbreur committed Jul 25, 2024
    Configuration menu
    Copy the full SHA
    3011f16 View commit details
    Browse the repository at this point in the history

  3. Update the GraphQL schema 

    Graphql schema updated, hopefully this is right because windows/wsl
version of `yarn run codegen` likes to break the schema sometimes.
    @daanbreur
    daanbreur committed Jul 25, 2024
    Configuration menu
    Copy the full SHA
    00f3b47 View commit details
    Browse the repository at this point in the history

  4. Add base implementation of register command. 

    This commit implements the base of some utils that are required and the base of the command itself.
    @daanbreur
    daanbreur committed Jul 25, 2024
    Configuration menu
    Copy the full SHA
    8b22bd2 View commit details
    Browse the repository at this point in the history

  5. Add checks to make sure user only has one token. 

    This commit implements fixes for the issues talked about in comment: TFNS#301 (comment)
Also changed the migration back to unique
    @daanbreur
    daanbreur committed Jul 25, 2024
    Configuration menu
    Copy the full SHA
    6d44414 View commit details
    Browse the repository at this point in the history

  6. Add ENV vars for configuring register command. 

    Adds the Environment variables that are needed to configure the register
command and some explanation for them.
    @daanbreur
    daanbreur committed Jul 25, 2024
    Configuration menu
    Copy the full SHA
    59f64ff View commit details
    Browse the repository at this point in the history

  7. Adds checks to register command and some cleanup 

    Some general cleanup and fixes of todo's.
Also implemented the required checks and used data from the environment variables.
This command implements what showcased in issue message: TFNS#301 (comment)
    @daanbreur
    daanbreur committed Jul 25, 2024
    Configuration menu
    Copy the full SHA
    841941d View commit details
    Browse the repository at this point in the history

Commits on Aug 4, 2024

  1. Add role sync after /register command 

    With a setTimeout we can run code after the request lifecycle.
This is ugly, but it works. See also the comment in the code about this.

Solves the problem for TFNS#301 (comment)
    @JJ-8
    JJ-8 committed Aug 4, 2024
    Configuration menu
    Copy the full SHA
    7a27254 View commit details
    Browse the repository at this point in the history

  2. Refactor AllowedRoles to enum of strings

    @JJ-8
    JJ-8 committed Aug 4, 2024
    Configuration menu
    Copy the full SHA
    192cc38 View commit details
    Browse the repository at this point in the history

  3. Rename variables and change signature of functions 

    Variable renaming for consistency.
The `getInvitationUrl` should return null in case of error instead of an empty string in order clarify error state.
    @JJ-8
    JJ-8 committed Aug 4, 2024
    Configuration menu
    Copy the full SHA
    950afdd View commit details
    Browse the repository at this point in the history

  4. Move the registerWithToken role sync to the after mutation hook 

    It makes more sense to have it there. It was by accident placed in the before hook previously.
    @JJ-8
    JJ-8 committed Aug 4, 2024
    Configuration menu
    Copy the full SHA
    854dfcd View commit details
    Browse the repository at this point in the history

  5. Improve Discord messages text

    @JJ-8
    JJ-8 committed Aug 4, 2024
    Configuration menu
    Copy the full SHA
    640ee90 View commit details
    Browse the repository at this point in the history

  6. Add strict 'true' check for Discord registration

    @JJ-8
    JJ-8 committed Aug 4, 2024
    Configuration menu
    Copy the full SHA
    e7e6842 View commit details
    Browse the repository at this point in the history

  7. Small text improvements

    @JJ-8
    JJ-8 committed Aug 4, 2024
    Configuration menu
    Copy the full SHA
    eaeaea3 View commit details
    Browse the repository at this point in the history

  8. Improve /register output even more

    @JJ-8
    JJ-8 committed Aug 4, 2024
    Configuration menu
    Copy the full SHA
    ee44665 View commit details
    Browse the repository at this point in the history

  9. Add comments to sql migration file

    @daanbreur
    daanbreur authored Aug 4, 2024
    Configuration menu
    Copy the full SHA
    0644256 View commit details
    Browse the repository at this point in the history

  10. Rename registration logic function to match command name

    @daanbreur
    daanbreur committed Aug 4, 2024
    Configuration menu
    Copy the full SHA
    d5053c2 View commit details
    Browse the repository at this point in the history

  11. format fix

    @daanbreur
    daanbreur committed Aug 4, 2024
    Configuration menu
    Copy the full SHA
    22cdd89 View commit details
    Browse the repository at this point in the history