Skip to content

DEV-933: updates#233

Merged
mitjacotic merged 1 commit intomainfrom
feat-DEV-933
Apr 24, 2024
Merged

DEV-933: updates#233
mitjacotic merged 1 commit intomainfrom
feat-DEV-933

Conversation

@mitjacotic
Copy link
Copy Markdown
Collaborator

@mitjacotic mitjacotic commented Mar 11, 2024 •
edited
Loading

  • lint
  • do not create exchange for non internal scope
  • add publishToUser method overriding msg's exchange
  • update logging, remove log.verbose()
  • add tests (vite)
  • use native uuid, add build action
  • publish action
  • add mdc argument/class, better logging

@mitjacotic mitjacotic changed the title updates DEV-933: updates Mar 11, 2024
@mitjacotic mitjacotic marked this pull request as draft March 11, 2024 21:49
@mitjacotic mitjacotic force-pushed the feat-DEV-933 branch 2 times, most recently from c36762d to e9d111c Compare March 11, 2024 22:34
tomazic89
tomazic89 previously approved these changes Mar 15, 2024
View reviewed changes
asvetin
asvetin previously approved these changes Mar 15, 2024
View reviewed changes
@mitjacotic mitjacotic dismissed stale reviews from asvetin and tomazic89 via 3c582a2 March 18, 2024 13:37
@mitjacotic mitjacotic marked this pull request as ready for review April 24, 2024 13:46
@mitjacotic
updates:
98cb11c 
- lint
- do not create exchange for non internal scope
- add publishToUser method overriding msg's exchange
- update logging, remove log.verbose()
- add tests (vite)
- use native uuid, add build action
- publish action
- add mdc argument/class, better logging
@mitjacotic mitjacotic merged commit 50fba0a into main Apr 24, 2024
@mitjacotic mitjacotic deleted the feat-DEV-933 branch April 24, 2024 14:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ctomc ctomc ctomc approved these changes

@asvetin asvetin asvetin left review comments

@tomazic89 tomazic89 tomazic89 left review comments

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

@mitjacotic @ctomc @asvetin @tomazic89