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

chore: deploy staging #587

Merged
merged 43 commits into from
Jul 22, 2024
Merged

chore: deploy staging #587

merged 43 commits into from
Jul 22, 2024

Conversation

BlankParticle
Copy link
Member

What does this PR do?

Merges current main to stagging.
There are lot of changes and 2 db migrations.

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • Chore (refactoring code, technical debt, workflow improvements)
  • Enhancement (small improvements)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

Checklist

Required

  • Read Contributing Guide
  • Self-reviewed my own code
  • Tested my code in a local environment
  • Commented on my code in hard-to-understand areas
  • Checked for warnings, there are none
  • Removed all console.logs
  • Merged the latest changes from main onto my branch with git pull origin main
  • My changes don't cause any responsiveness issues

Appreciated

  • If a UI change was made: Added a screen recording or screenshots to this PR
  • Updated the UnInbox Docs if changes were necessary

McPizza0 and others added 30 commits July 1, 2024 17:32
@McPizza0
settings-org-billing (#528)
4c38254
@BlankParticle
chore: Replace radix-theme components across the repo (Part 1) (#526)
76a24d6
@BlankParticle
chore: Replace all radix-theme components (Part 2) (#527)
5b84daa
@BlankParticle
chore: Better health endpoints (#529)
5738b80
@BlankParticle
feat: Add worker app for scheduled/repeatable tasks (#530)
a6c3f91
@BlankParticle
fix: file name generation for missing filenames for attachments (#533)
4b63284
@BlankParticle
chore: update @hono/trpc-server to 0.3.2 (#534)
1a2ad3d
@BlankParticle
fix: replace convo list with Virtuoso component to fix bugs (#536)
9689eac
@BlankParticle
chore: add more database indexes for better performance (#535)
b319757 
## What does this PR do?

Add missing database indexes to id, public id and timestamps for better performance

## Type of change

<!-- Please mark the relevant points by using [x] -->

- [ ] Bug fix (non-breaking change which fixes an issue)
- [ ] Chore (refactoring code, technical debt, workflow improvements)
- [ ] Enhancement (small improvements)
- [ ] New feature (non-breaking change which adds functionality)
- [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
- [ ] This change requires a documentation update

## Checklist

<!-- We're starting to get more and more contributions. Please help us making this efficient for all of us and go through this checklist. Please tick off what you did  -->

### Required

- [ ] Read [Contributing Guide](https://github.com/un/inbox/blob/main/CONTRIBUTING.md)
- [ ] Self-reviewed my own code
- [ ] Tested my code in a local environment
- [ ] Commented on my code in hard-to-understand areas
- [ ] Checked for warnings, there are none
- [ ] Removed all `console.logs`
- [ ] Merged the latest changes from main onto my branch with `git pull origin main`
- [ ] My changes don't cause any responsiveness issues

### Appreciated

- [ ] If a UI change was made: Added a screen recording or screenshots to this PR
- [ ] Updated the UnInbox Docs if changes were necessary
@BlankParticle
fix: new email based participants are correctly added now (#538)
9df941d
@BlankParticle
feat: add turnstile to sign in, sign up, and recovery pages (#540)
80a21bb
@BlankParticle
feat: rewrite ratelimit to be more extensible (#541)
a1c34b6
@chrisdadev13
feat(next): add metadata to convo pages (#542)
aaade37 
## What does this PR do?

Hi everyone! This PR adds document titles and metadata to each page in the convo route indicating information about the `page/convo`. I closed the first PR to make it more focused, since applying this logic in the `settings` path would have implied too many changes. If this one is merged, I can open another one for the `settings` routes using the same logic.

Fixes #215

## Type of change

- [ ] Bug fix (non-breaking change which fixes an issue)
- [x] Chore (refactoring code, technical debt, workflow improvements)
- [ ] Enhancement (small improvements)
- [ ] New feature (non-breaking change which adds functionality)
- [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
- [ ] This change requires a documentation update

## Checklist

### Required

- [x] Read [Contributing Guide](https://github.com/un/inbox/blob/main/CONTRIBUTING.md)
- [x] Self-reviewed my own code
- [x] Tested my code in a local environment
- [ ] Commented on my code in hard-to-understand areas
- [x] Checked for warnings, there are none
- [x] Removed all `console.logs`
- [x] Merged the latest changes from main onto my branch with `git pull origin main`
- [x] My changes don't cause any responsiveness issues

### Appreciated

- [ ] If a UI change was made: Added a screen recording or screenshots to this PR
- [ ] Updated the UnInbox Docs if changes were necessary
@BlankParticle
fix: check org shortCode availability before creating org (#544)
0bba49c
@BlankParticle
feat: disable sending emails from unverified email identities (#545)
bfb6e6e
@yash-kh
fix: password auto complete suggestion fix (#546)
1ff21dc 
Co-authored-by: Yash Khatri <[email protected]>
@McPizza0 @BlankParticle
chore: Remove Nuxt web-app and related code (#551)
6ac6079 
* remove-nuxt

* fix: eslint config with typescript

* chore: vscode config cleanup

* chore: remove vue from tiptap

* chore: remove deprecated trpc routes

* chore: readme and config files cleanup

---------

Co-authored-by: BlankParticle <[email protected]>
@BlankParticle
feat: change domain DNS flow to new planned flow (#552)
dafe2fc
@BlankParticle
chore: setup custom eslint package for table-needs-org-id rule (#554)
127fef9
@BlankParticle
feat: web app env is now type safe and auto prefixed (#558)
eee9470
@BlankParticle
chore: update dependencies (#560)
957d606
@rockingrohit9639
fix(signup): save profile without avatar (#561)
29fbf7f 
## What does this PR do?

🖤

When signing up, allow user to save profile without uploading avatar

Fixes #549 
<!-- If there isn't an issue for this PR, please re-review our Contributing Guide and create an issue -->

## Type of change

<!-- Please mark the relevant points by using [x] -->

- [x] Bug fix (non-breaking change which fixes an issue)
- [ ] Chore (refactoring code, technical debt, workflow improvements)
- [ ] Enhancement (small improvements)
- [ ] New feature (non-breaking change which adds functionality)
- [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
- [ ] This change requires a documentation update

## Checklist

<!-- We're starting to get more and more contributions. Please help us making this efficient for all of us and go through this checklist. Please tick off what you did  -->

### Required

- [x] Read [Contributing Guide](https://github.com/un/inbox/blob/main/CONTRIBUTING.md)
- [x] Self-reviewed my own code
- [x] Tested my code in a local environment
- [x] Commented on my code in hard-to-understand areas
- [x] Checked for warnings, there are none
- [x] Removed all `console.logs`
- [x] Merged the latest changes from main onto my branch with `git pull origin main`
- [x] My changes don't cause any responsiveness issues
@BlankParticle
feat: org issues dialog for org admins (#547)
dbaabd8
@BlankParticle
fix: make trpc redirects work in web (#555)
025562f 
## What does this PR do?

trpc can properly throw a redirect on web with `Location` header now.

## Type of change

<!-- Please mark the relevant points by using [x] -->

- [ ] Bug fix (non-breaking change which fixes an issue)
- [ ] Chore (refactoring code, technical debt, workflow improvements)
- [ ] Enhancement (small improvements)
- [ ] New feature (non-breaking change which adds functionality)
- [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
- [ ] This change requires a documentation update

## Checklist

<!-- We're starting to get more and more contributions. Please help us making this efficient for all of us and go through this checklist. Please tick off what you did  -->

### Required

- [ ] Read [Contributing Guide](https://github.com/un/inbox/blob/main/CONTRIBUTING.md)
- [ ] Self-reviewed my own code
- [ ] Tested my code in a local environment
- [ ] Commented on my code in hard-to-understand areas
- [ ] Checked for warnings, there are none
- [ ] Removed all `console.logs`
- [ ] Merged the latest changes from main onto my branch with `git pull origin main`
- [ ] My changes don't cause any responsiveness issues

### Appreciated

- [ ] If a UI change was made: Added a screen recording or screenshots to this PR
- [ ] Updated the UnInbox Docs if changes were necessary
@BlankParticle
feat: improve middleware performance by using plain fetch calls (#563)
1d2a050
@BlankParticle
feat: experimental standalone build mode
3fb29b0
@yash-kh
Hiding/un-hiding convo fix (#565)
f95a514 
## What does this PR do?

Hiding/un-hiding convo fix

Fixes #564 

## Type of change

- [x] Bug fix (non-breaking change which fixes an issue)
- [ ] Chore (refactoring code, technical debt, workflow improvements)
- [ ] Enhancement (small improvements)
- [ ] New feature (non-breaking change which adds functionality)
- [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
- [ ] This change requires a documentation update
@McPizza0
icanhaz (#569)
5ae63f7
@yash-kh
When fetching a conversation, update lastReadAt (#562)
1794e60
@Esther-Lita
Rename tRPC client to platform instead of api (#571)
3fd44f5
BlankParticle and others added 13 commits July 17, 2024 19:21
@BlankParticle
fix: rename the postal local mode env variable to be proper one (#573)
8d71947 
## What does this PR do?

The name of the variable and the in code was mismatched

<!-- If there isn't an issue for this PR, please re-review our Contributing Guide and create an issue -->

## Type of change

<!-- Please mark the relevant points by using [x] -->

- [ ] Bug fix (non-breaking change which fixes an issue)
- [ ] Chore (refactoring code, technical debt, workflow improvements)
- [ ] Enhancement (small improvements)
- [ ] New feature (non-breaking change which adds functionality)
- [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
- [ ] This change requires a documentation update

## Checklist

<!-- We're starting to get more and more contributions. Please help us making this efficient for all of us and go through this checklist. Please tick off what you did  -->

### Required

- [ ] Read [Contributing Guide](https://github.com/un/inbox/blob/main/CONTRIBUTING.md)
- [ ] Self-reviewed my own code
- [ ] Tested my code in a local environment
- [ ] Commented on my code in hard-to-understand areas
- [ ] Checked for warnings, there are none
- [ ] Removed all `console.logs`
- [ ] Merged the latest changes from main onto my branch with `git pull origin main`
- [ ] My changes don't cause any responsiveness issues

### Appreciated

- [ ] If a UI change was made: Added a screen recording or screenshots to this PR
- [ ] Updated the UnInbox Docs if changes were necessary
@McPizza0
add-gh-autofix (#576)
20e84d8
@yash-kh @BlankParticle
select z index fix and input box default length fix (#570)
32dced5 
* select z index fix and input box default length fix

* z-index for shadcn-ui updated to default

* fix: sidebar expand button overlap

---------

Co-authored-by: BlankParticle <[email protected]>
@Esther-Lita
Created the orgAdminProcedure (#574)
b50a94f 
Created a procedure to separate the logic checking if the user  in the organization is an administrator
@benjaminshafii
fix-redirect-when-already-in-org (#578)
f730f55 
## What does this PR do?

<!-- 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. -->

Fixes # (issue)

_If there is not an issue for this, please create one first. This is used to tracking purposes and also helps use understand why this PR exists_

<!-- If there isn't an issue for this PR, please re-review our Contributing Guide and create an issue -->

## Type of change

<!-- Please mark the relevant points by using [x] -->

- [x] Bug fix (non-breaking change which fixes an issue)
- [ ] Chore (refactoring code, technical debt, workflow improvements)
- [ ] Enhancement (small improvements)
- [ ] New feature (non-breaking change which adds functionality)
- [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
- [ ] This change requires a documentation update

## Checklist

<!-- We're starting to get more and more contributions. Please help us making this efficient for all of us and go through this checklist. Please tick off what you did  -->

### Required

- [ ] Read [Contributing Guide](https://github.com/un/inbox/blob/main/CONTRIBUTING.md)
- [x] Self-reviewed my own code
- [x] Tested my code in a local environment
- [ ] Commented on my code in hard-to-understand areas
- [ ] Checked for warnings, there are none
- [ ] Removed all `console.logs`
- [ ] Merged the latest changes from main onto my branch with `git pull origin main`
- [ ] My changes don't cause any responsiveness issues

### Appreciated

- [ ] If a UI change was made: Added a screen recording or screenshots to this PR
- [ ] Updated the UnInbox Docs if changes were necessary
@McPizza0
fix-rename-shortCode-shortcode (#580)
7d10378 
* fix-rename-shortCode-shortcode

* fix-rename-shortCode-shortcode

* feat-spaces

* rename back
@benjaminshafii
feat-display-current-theme (#581)
2393f6a
@BlankParticle
feat: build apps with tsup and use js files on production (#579)
f23b8ca
@BlankParticle
feat: hono package and helpers are now centralized in @u22n/hono (#583
317a05d 
)
@BlankParticle
feat: new sign up UI (#575)
b9813d4
@BlankParticle
feat: setup optional OpenTelemetry (#585)
471e56f 
## What does this PR do?

Open Telemetry setup with No-op when not enabled.
fixes #507 

## Type of change

<!-- Please mark the relevant points by using [x] -->

- [ ] Bug fix (non-breaking change which fixes an issue)
- [ ] Chore (refactoring code, technical debt, workflow improvements)
- [ ] Enhancement (small improvements)
- [ ] New feature (non-breaking change which adds functionality)
- [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
- [ ] This change requires a documentation update

## Checklist

<!-- We're starting to get more and more contributions. Please help us making this efficient for all of us and go through this checklist. Please tick off what you did  -->

### Required

- [ ] Read [Contributing Guide](https://github.com/un/inbox/blob/main/CONTRIBUTING.md)
- [ ] Self-reviewed my own code
- [ ] Tested my code in a local environment
- [ ] Commented on my code in hard-to-understand areas
- [ ] Checked for warnings, there are none
- [ ] Removed all `console.logs`
- [ ] Merged the latest changes from main onto my branch with `git pull origin main`
- [ ] My changes don't cause any responsiveness issues

### Appreciated

- [ ] If a UI change was made: Added a screen recording or screenshots to this PR
- [ ] Updated the UnInbox Docs if changes were necessary
@BlankParticle
hotfix: OTEL env variables issue
133039e
@BlankParticle
hotfix: Web app variables issue
ca84b79
@BlankParticle BlankParticle changed the title chore: deploy stagging chore: deploy staging Jul 22, 2024
@McPizza0 McPizza0 merged commit bab6520 into staging Jul 22, 2024
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@McPizza0 McPizza0 Awaiting requested review from McPizza0 McPizza0 is a code owner

Assignees

@BlankParticle BlankParticle

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@BlankParticle @McPizza0 @chrisdadev13 @yash-kh @rockingrohit9639 @Esther-Lita @benjaminshafii