inbox icon indicating copy to clipboard operation
inbox copied to clipboard

Rename org short code feature added

Open yash-kh opened this issue 1 year ago • 7 comments

What does this PR do?

Rename org short code Option added in: settings -> Organization -> Org profile

Fixes #307

Type of change

  • [ ] Bug fix (non-breaking change which fixes an issue)
  • [ ] Chore (refactoring code, technical debt, workflow improvements)
  • [ ] Enhancement (small improvements)
  • [x] 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

yash-kh avatar Jul 09 '24 10:07 yash-kh

Graphite Automations

"Auto-assign PRs to author" took an action on this PR • (07/09/24)

1 assignee was added to this PR based on Rahul Mishra's automation.

graphite-app[bot] avatar Jul 09 '24 10:07 graphite-app[bot]

there are lot of stuff tied to org short code, if we want to change org short code we need a list of burnt orgshort codes and make older codes redirect to new codes.

Invalidating cache is not a big issue tho

That also means every query would need to check if org shortcode has been redirected before proceding

I think we should delay this feature until we have a good flow planned

BlankParticle avatar Jul 10 '24 12:07 BlankParticle

I checked some cases, if the user uses the URL with the old code it redirects to: Screenshot 2024-07-10 at 6 08 49 PM

And yeah, the old code still works for the active user it's not updating automatically the inbox and conversations are opening fine. Upon refresh, its same error.

yash-kh avatar Jul 10 '24 12:07 yash-kh

Also, the orgProcedure uses shortCode which will be an issue.

yash-kh avatar Jul 10 '24 12:07 yash-kh

@BlankParticle I dont think we need to burn shortcodes the same way we burn usernames but we do need a way to safely redirect logged in users to their new org shortcode the realtime system is not viable for this

we already have a method to invalidate an org's shortcode cache but again that could mean if a user is in the middle of writing an email, they would end up getting errors and losing the email they wrote.

OmarMcAdam avatar Jul 10 '24 13:07 OmarMcAdam

@yash-kh 99% of queries use the orgProcedure method, so we can class it as "it would break everything"

OmarMcAdam avatar Jul 10 '24 13:07 OmarMcAdam

I think we should put this in backlog for now

BlankParticle avatar Jul 10 '24 14:07 BlankParticle