Skip to content

Update notification types in Listing Creation#2837

Open
AndersOlsson-Sekoia wants to merge 1 commit intomainfrom
AndersOlsson-Sekoia-patch-1
Open

Update notification types in Listing Creation#2837
AndersOlsson-Sekoia wants to merge 1 commit intomainfrom
AndersOlsson-Sekoia-patch-1

Conversation

@AndersOlsson-Sekoia
Copy link
Contributor

Added new notification types and updated error handling.

Added new notification types and updated error handling.
@github-actions
Copy link

Newest code from AndersOlsson-Sekoia has been published to preview environment

🚀 Latest deployment was built on 2026-02-27 09:46:52 (0cb4cb6a039ddd473843318cb05edb0409538cb7).

@alexane-bougeardbebin-sekoia
Copy link
Contributor

Hi @AndersOlsson-Sekoia

Thank you for your draft. Here are some suggestions to enhance the article:

  1. It is unclear what a notification will be (mail, sms, message in slack, pop-up,... ) from the start so I don't know if I am interested in setting this up until I read everything up to step 3 of creating a notification.
  2. The path is a bit unclear to access it:
  • "Notifications, accessible through the main menu or Settings" (where is Settings? What really is the main menu? A screenshot will be really helpful here (or at least describe the path ("bottom of the left-hand panel" for the settings button) Will I be directly in the right tabs of settings, or should I click somewhere else?
  • "The listing page will be blank with a highlighted' What is the listing page? Where is it? At some point, you should either specify the path or tell before that the path you are providing will redirect to "Listing page"
  1. For me, we should have 3 articles:
  • 1 concept with the capacity (first 32 rows + single tenant/multi-tenant mode)
  • 1 task: creation + example + link to example + link to third article
  • 1 task with management: (you say that you can edit, delete, etc, but you are not telling how. I agree it's pretty straightforward but having a dedicated article might help some users) It could be a good place for some troubleshoot/FAQ (if we don't have too many) : can I unsuscribe a notification I haven't created? Etc
  1. There are some "style issues:
  • 2 step 1 in creation,
  • no ending punctuation,
  • some info should be a call-out/admonitions block
  1. The example is nice but this is the solution only; we don't have the problem it solves, nor the result so it's hard to leverage it to get inspiration and understand the logic before creating something else autonomously.

I can help you with that and or you can ask our assistant to help you slice it and provide raw materials to write the 3rd article, but I think it's beneficial to explain the logic behind it before relying on others or AI ;)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants