Skip to content

Conversation

@st3iny
Copy link
Member

@st3iny st3iny commented Apr 7, 2021

Fixes #4871

Move envelope tags to global object and index them by their id. Every envelope will store an array of tag ids in envelope.tags.

Get all tag objects of an envelope via $store.getters.getEnvelopeTags(envelope.databaseId)
Add a tag to an envelope via $store.dispatch('addEnvelopeTag', { envelope, imapLabel })
Remove a tag from an envelope via $store.dispatch('removeEnvelopeTag', { envelope, tag })
Get a tag from the global store via $store.getters.getTag(tagId)

This PR will also fix the important toggles.

@st3iny st3iny self-assigned this Apr 7, 2021
@st3iny st3iny force-pushed the enhancement/4871/normalize-tags-in-vuex branch 2 times, most recently from 96d740a to c67a1b3 Compare April 8, 2021 09:56
@st3iny st3iny marked this pull request as ready for review April 8, 2021 18:21
Copy link
Member

@ChristophWurst ChristophWurst left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code looks excellent. Tested important flag handling a bit and everything seems to work as expected :shipit:

Signed-off-by: Richard Steinmetz <richard@steinmetz.cloud>
@ChristophWurst ChristophWurst force-pushed the enhancement/4871/normalize-tags-in-vuex branch from f1e1509 to d4e4817 Compare April 9, 2021 12:34
@ChristophWurst ChristophWurst added this to the v1.10.0 milestone Apr 9, 2021
@ChristophWurst ChristophWurst merged commit 4e75585 into master Apr 9, 2021
@ChristophWurst ChristophWurst deleted the enhancement/4871/normalize-tags-in-vuex branch April 9, 2021 12:54
@ChristophWurst ChristophWurst mentioned this pull request May 19, 2021
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Normalize message/envelope tags in the Vuex store

3 participants