Skip to content

change invite to disc#799

Merged
brarsanmol merged 2 commits intodevfrom
change_slack_to_disc_invite
Jan 21, 2022
Merged

change invite to disc#799
brarsanmol merged 2 commits intodevfrom
change_slack_to_disc_invite

Conversation

@pengyk
Copy link
Member

@pengyk pengyk commented Jan 21, 2022

Tickets:

  • HAC-

List of changes:

Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • New release
  • This change requires a documentation update

How has this been tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

  • Test A
  • Test B

Test Configuration:

Firmware version:
Hardware:
Toolchain:
SDK:

Questions for code reviewers?

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • My changes generate no new warnings
  • Listed change(s) in the Changelog
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • I have made corresponding changes to the documentation
  • Any dependent changes have been merged and published in downstream modules

@pengyk pengyk requested a review from brarsanmol January 21, 2022 09:21
@brarsanmol brarsanmol changed the base branch from dev to master January 21, 2022 17:35
@brarsanmol brarsanmol changed the base branch from master to dev January 21, 2022 17:35
@brarsanmol brarsanmol merged commit efc77e3 into dev Jan 21, 2022
@brarsanmol brarsanmol deleted the change_slack_to_disc_invite branch January 21, 2022 17:35
brarsanmol added a commit that referenced this pull request Jan 21, 2022
@brarsanmol @meldunn @pengyk
chore: merge dev to main. (#800)
2a12f43 
* feat(hacker.js,hacker.middleware.js): add ability for discord verification. (#795)

* feat(hacker.js,hacker.middleware.js): add ability for discord verification..

* fix(hacker.js): typo in middleware function call.

* add postDiscord role/route

* Fix hacker discord route

Co-authored-by: meldunn <dmelissa216@gmail.com>

* change invite to disc (#799)

* change invite to disc

* remove fb group

Co-authored-by: meldunn <dmelissa216@gmail.com>
Co-authored-by: Yun Kai Peng <45922265+pengyk@users.noreply.github.com>
brarsanmol added a commit that referenced this pull request Jan 17, 2023
@brarsanmol @meldunn
@pengyk @jackyyzhang03
chore: merge dev to main (#902)
3baff4d 
* feat(hacker.js,hacker.middleware.js): add ability for discord verification. (#795)

* feat(hacker.js,hacker.middleware.js): add ability for discord verification..

* fix(hacker.js): typo in middleware function call.

* add postDiscord role/route

* Fix hacker discord route

Co-authored-by: meldunn <dmelissa216@gmail.com>

* change invite to disc (#799)

* change invite to disc

* remove fb group

* fix/update emails (#882)

* feat(hacker.js,hacker.middleware.js): add ability for discord verification..

* fix(hacker.js): typo in middleware function call.

* add postDiscord role/route

* fix: update applied email

Co-authored-by: meldunn <dmelissa216@gmail.com>

* Jacky/email (#900)

* fix: change to mchacks 10 in email templates

* fix: dates

* fix: move confirmation date +1 day

Co-authored-by: Jacky Zhang <jackyyzhang@gmail.com>

Co-authored-by: meldunn <dmelissa216@gmail.com>
Co-authored-by: Yun Kai Peng <45922265+pengyk@users.noreply.github.com>
Co-authored-by: Jacky Zhang <jackyyzhang@gmail.com>
brarsanmol added a commit that referenced this pull request Jan 28, 2023
@brarsanmol @meldunn
@pengyk @jackyyzhang03
chore: dev to release (#906)
42c5c48 
* feat(hacker.js,hacker.middleware.js): add ability for discord verification. (#795)

* feat(hacker.js,hacker.middleware.js): add ability for discord verification..

* fix(hacker.js): typo in middleware function call.

* add postDiscord role/route

* Fix hacker discord route

Co-authored-by: meldunn <dmelissa216@gmail.com>

* change invite to disc (#799)

* change invite to disc

* remove fb group

* fix/update emails (#882)

* feat(hacker.js,hacker.middleware.js): add ability for discord verification..

* fix(hacker.js): typo in middleware function call.

* add postDiscord role/route

* fix: update applied email

Co-authored-by: meldunn <dmelissa216@gmail.com>

* Jacky/email (#900)

* fix: change to mchacks 10 in email templates

* fix: dates

* fix: move confirmation date +1 day

Co-authored-by: Jacky Zhang <jackyyzhang@gmail.com>

* Fix/check in discord (#905)

* fix: change to mchacks 10 in email templates

* fix: dates

* fix: move confirmation date +1 day

* Fix: check in discord

---------

---------

Co-authored-by: meldunn <dmelissa216@gmail.com>
Co-authored-by: Yun Kai Peng <45922265+pengyk@users.noreply.github.com>
Co-authored-by: Jacky Zhang <jackyyzhang@gmail.com>
jackyyzhang03 added a commit that referenced this pull request Nov 15, 2023
@jackyyzhang03 @brarsanmol
@meldunn @pengyk
chore: dev to master (#923)
227d3f0 
* feat(hacker.js,hacker.middleware.js): add ability for discord verification. (#795)

* feat(hacker.js,hacker.middleware.js): add ability for discord verification..

* fix(hacker.js): typo in middleware function call.

* add postDiscord role/route

* Fix hacker discord route

Co-authored-by: meldunn <dmelissa216@gmail.com>

* change invite to disc (#799)

* change invite to disc

* remove fb group

* fix/update emails (#882)

* feat(hacker.js,hacker.middleware.js): add ability for discord verification..

* fix(hacker.js): typo in middleware function call.

* add postDiscord role/route

* fix: update applied email

Co-authored-by: meldunn <dmelissa216@gmail.com>

* Jacky/email (#900)

* fix: change to mchacks 10 in email templates

* fix: dates

* fix: move confirmation date +1 day

Co-authored-by: Jacky Zhang <jackyyzhang@gmail.com>

* Fix/check in discord (#905)

* fix: change to mchacks 10 in email templates

* fix: dates

* fix: move confirmation date +1 day

* Fix: check in discord

---------

* feat: add field for reimbursement reason (#922)

* feat: set cookie SameSite using env variable

---------

Co-authored-by: Anmol Brar <83686967+brarsanmol@users.noreply.github.com>
Co-authored-by: meldunn <dmelissa216@gmail.com>
Co-authored-by: Yun Kai Peng <45922265+pengyk@users.noreply.github.com>
jackyyzhang03 added a commit that referenced this pull request Dec 21, 2023
@jackyyzhang03 @brarsanmol
@meldunn @pengyk
chore: dev to master (#924)
267f13b 
* feat(hacker.js,hacker.middleware.js): add ability for discord verification. (#795)

* feat(hacker.js,hacker.middleware.js): add ability for discord verification..

* fix(hacker.js): typo in middleware function call.

* add postDiscord role/route

* Fix hacker discord route

Co-authored-by: meldunn <dmelissa216@gmail.com>

* change invite to disc (#799)

* change invite to disc

* remove fb group

* fix/update emails (#882)

* feat(hacker.js,hacker.middleware.js): add ability for discord verification..

* fix(hacker.js): typo in middleware function call.

* add postDiscord role/route

* fix: update applied email

Co-authored-by: meldunn <dmelissa216@gmail.com>

* Jacky/email (#900)

* fix: change to mchacks 10 in email templates

* fix: dates

* fix: move confirmation date +1 day

Co-authored-by: Jacky Zhang <jackyyzhang@gmail.com>

* Fix/check in discord (#905)

* fix: change to mchacks 10 in email templates

* fix: dates

* fix: move confirmation date +1 day

* Fix: check in discord

---------

* feat: add field for reimbursement reason (#922)

* feat: set cookie SameSite using env variable

* feat: update status update emails

---------

Co-authored-by: Anmol Brar <83686967+brarsanmol@users.noreply.github.com>
Co-authored-by: meldunn <dmelissa216@gmail.com>
Co-authored-by: Yun Kai Peng <45922265+pengyk@users.noreply.github.com>
jackyyzhang03 added a commit that referenced this pull request Jan 17, 2024
@jackyyzhang03 @brarsanmol
@meldunn @pengyk
chore: dev to master (#927)
c4eac3c 
* feat(hacker.js,hacker.middleware.js): add ability for discord verification. (#795)

* feat(hacker.js,hacker.middleware.js): add ability for discord verification..

* fix(hacker.js): typo in middleware function call.

* add postDiscord role/route

* Fix hacker discord route

Co-authored-by: meldunn <dmelissa216@gmail.com>

* change invite to disc (#799)

* change invite to disc

* remove fb group

* fix/update emails (#882)

* feat(hacker.js,hacker.middleware.js): add ability for discord verification..

* fix(hacker.js): typo in middleware function call.

* add postDiscord role/route

* fix: update applied email

Co-authored-by: meldunn <dmelissa216@gmail.com>

* Jacky/email (#900)

* fix: change to mchacks 10 in email templates

* fix: dates

* fix: move confirmation date +1 day

Co-authored-by: Jacky Zhang <jackyyzhang@gmail.com>

* Fix/check in discord (#905)

* fix: change to mchacks 10 in email templates

* fix: dates

* fix: move confirmation date +1 day

* Fix: check in discord

---------

* feat: add field for reimbursement reason (#922)

* feat: set cookie SameSite using env variable

* feat: update status update emails

* feat: remove facebook and twitter from status emails

---------

Co-authored-by: Anmol Brar <83686967+brarsanmol@users.noreply.github.com>
Co-authored-by: meldunn <dmelissa216@gmail.com>
Co-authored-by: Yun Kai Peng <45922265+pengyk@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@brarsanmol brarsanmol Awaiting requested review from brarsanmol

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@pengyk @brarsanmol

Comments