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

📩 feat: invite user #3012

Merged
merged 22 commits into from
Aug 18, 2024
Merged

📩 feat: invite user #3012

merged 22 commits into from
Aug 18, 2024

Conversation

berry-13
Copy link
Collaborator

@berry-13 berry-13 commented Jun 8, 2024 •
edited
Loading

Summary

feat: invite user

invite a user with the invite-user script and they will recive an email with the link.
If SMTP it's not enabled, the script will return the link in the console log

Change Type

  • New feature (non-breaking change which adds functionality)

Testing

Tested with Registration enabled and disabled

Checklist

  • My code adheres to this project's style guidelines
  • I have performed a self-review of my own code
  • I have commented in any complex areas of my code
  • I have made pertinent documentation changes
  • My changes do not introduce new warnings
  • I have written tests demonstrating that my changes are effective or that my feature works
  • Local unit tests pass with my changes
  • Any changes dependent on mine have been merged and published in downstream modules.
  • A pull request for updating the documentation has been submitted.

danny-avila
danny-avila requested changes Jun 8, 2024
View reviewed changes
api/models/inviteUser.js Show resolved Hide resolved
api/server/middleware/checkInviteUser.js Outdated Show resolved Hide resolved
api/server/middleware/validateRegistration.js Outdated Show resolved Hide resolved
@berry-13 berry-13 marked this pull request as ready for review June 9, 2024 20:59
@berry-13 berry-13 mentioned this pull request Jun 29, 2024
Closed
1 task
@danny-avila danny-avila merged commit bbb9324 into main Aug 18, 2024
4 checks passed
@danny-avila danny-avila deleted the invite-user branch August 18, 2024 04:23
danny-avila added a commit that referenced this pull request Aug 18, 2024
@berry-13 @danny-avila
📩 feat: invite user (#3012)
3d0e779 
* feat: basic invite-user script

* feat: add invite user functionality and registration validation middleware

* fix: invite user fixes

* refactor: consolidate direct model access to a central place of functions

* style(Registration): add spinner to continue button

* refactor: import ordrer

* feat: improve invite user script and error handling

* fix: merge conflict

* refactor: remove `console.log` and use `logger`

* fix: token operation and checkinvite issues

* bring back comment and remove console log

* fix: return invalid token when token is not found

* fix: getInvite fix

* refactor: Update Token.js to use async/await syntax for update and delete operations

* feat: Refactor Token.js to use async/await syntax for createToken and findToken functions

* refactor(inviteUser): define functions outside of module.exports

* Update AuthService.js

---------

Co-authored-by: Danny Avila <danny@librechat.ai>
berry-13 added a commit that referenced this pull request Aug 18, 2024
@danny-avila @kmeshcheryakov
@berry-13 @bsu3338
🧪 feat: Prompt Dropdown Variable; style: Add Markdown Support (#3681)
d3a2035 
* feat: Add extended inputs for promts library variables

* feat: Add maxRows prop to VariableForm input field

* 📩 feat: invite user (#3012)

* feat: basic invite-user script

* feat: add invite user functionality and registration validation middleware

* fix: invite user fixes

* refactor: consolidate direct model access to a central place of functions

* style(Registration): add spinner to continue button

* refactor: import ordrer

* feat: improve invite user script and error handling

* fix: merge conflict

* refactor: remove `console.log` and use `logger`

* fix: token operation and checkinvite issues

* bring back comment and remove console log

* fix: return invalid token when token is not found

* fix: getInvite fix

* refactor: Update Token.js to use async/await syntax for update and delete operations

* feat: Refactor Token.js to use async/await syntax for createToken and findToken functions

* refactor(inviteUser): define functions outside of module.exports

* Update AuthService.js

---------

Co-authored-by: Danny Avila <danny@librechat.ai>

* style: improve OpenAI.tsx input field focus styling

* refactor: update import statement in Input.tsx

* refactor: remove multi-line

* refactor:  update placeholder text to use localization

* style: new dropdown variable info and markdown styling for info

* Add ReactMarkdown

* chore: styling, import order

* refactor: update ReactMarkdown usage in VariableForm

* style: remove markdown class

* refactor: update mobile styling and use code renderer

* style(InputWithDropDown): update focus trigger style

* style(OptionsPopover): update Save As Preset `focus` and `dark:bg`

---------

Co-authored-by: Konstantin Meshcheryakov <kmeshcheryakov@klika-tech.com>
Co-authored-by: Marco Beretta <81851188+berry-13@users.noreply.github.com>
Co-authored-by: bsu3338 <bsu3338@users.noreply.github.com>
@berry-13 berry-13 mentioned this pull request Aug 19, 2024
Merged
10 tasks
@fbaeumer
Copy link

fbaeumer commented Sep 4, 2024

It would be great to have an option to provide some balance with the invitation!

@BennyMorton28
Copy link

BennyMorton28 commented Sep 11, 2024 •
edited
Loading

This is cool. How do we actually invite the user/is there a button? @berry-13

kenshinsamue pushed a commit to intelequia/LibreChat that referenced this pull request Sep 17, 2024
@berry-13 @danny-avila
📩 feat: invite user (danny-avila#3012)
74ce382 
* feat: basic invite-user script

* feat: add invite user functionality and registration validation middleware

* fix: invite user fixes

* refactor: consolidate direct model access to a central place of functions

* style(Registration): add spinner to continue button

* refactor: import ordrer

* feat: improve invite user script and error handling

* fix: merge conflict

* refactor: remove `console.log` and use `logger`

* fix: token operation and checkinvite issues

* bring back comment and remove console log

* fix: return invalid token when token is not found

* fix: getInvite fix

* refactor: Update Token.js to use async/await syntax for update and delete operations

* feat: Refactor Token.js to use async/await syntax for createToken and findToken functions

* refactor(inviteUser): define functions outside of module.exports

* Update AuthService.js

---------

Co-authored-by: Danny Avila <danny@librechat.ai>
kenshinsamue pushed a commit to intelequia/LibreChat that referenced this pull request Sep 17, 2024
@danny-avila @kmeshcheryakov
@berry-13 @bsu3338
🧪 feat: Prompt Dropdown Variable; style: Add Markdown Support (danny-…
fe1eed2 
…avila#3681)

* feat: Add extended inputs for promts library variables

* feat: Add maxRows prop to VariableForm input field

* 📩 feat: invite user (danny-avila#3012)

* feat: basic invite-user script

* feat: add invite user functionality and registration validation middleware

* fix: invite user fixes

* refactor: consolidate direct model access to a central place of functions

* style(Registration): add spinner to continue button

* refactor: import ordrer

* feat: improve invite user script and error handling

* fix: merge conflict

* refactor: remove `console.log` and use `logger`

* fix: token operation and checkinvite issues

* bring back comment and remove console log

* fix: return invalid token when token is not found

* fix: getInvite fix

* refactor: Update Token.js to use async/await syntax for update and delete operations

* feat: Refactor Token.js to use async/await syntax for createToken and findToken functions

* refactor(inviteUser): define functions outside of module.exports

* Update AuthService.js

---------

Co-authored-by: Danny Avila <danny@librechat.ai>

* style: improve OpenAI.tsx input field focus styling

* refactor: update import statement in Input.tsx

* refactor: remove multi-line

* refactor:  update placeholder text to use localization

* style: new dropdown variable info and markdown styling for info

* Add ReactMarkdown

* chore: styling, import order

* refactor: update ReactMarkdown usage in VariableForm

* style: remove markdown class

* refactor: update mobile styling and use code renderer

* style(InputWithDropDown): update focus trigger style

* style(OptionsPopover): update Save As Preset `focus` and `dark:bg`

---------

Co-authored-by: Konstantin Meshcheryakov <kmeshcheryakov@klika-tech.com>
Co-authored-by: Marco Beretta <81851188+berry-13@users.noreply.github.com>
Co-authored-by: bsu3338 <bsu3338@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@danny-avila danny-avila danny-avila requested changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@berry-13 @fbaeumer @BennyMorton28 @danny-avila