Differential D3465

Import Postman collection for Notifications Delivery API
ClosedPublic
Actions

Authored by dereckson on Sep 25 2024, 21:41.

Tags

None

Referenced Files

	F3927472: D3465.id.diff
	Sun, Dec 22, 00:26

	F3927410: D3465.diff
	Sun, Dec 22, 00:06

	Unknown Object (File)
	Mon, Dec 9, 20:03

	Unknown Object (File)
	Fri, Dec 6, 15:55

	Unknown Object (File)
	Fri, Dec 6, 13:42

	Unknown Object (File)
	Wed, Dec 4, 10:36

	Unknown Object (File)
	Sun, Nov 24, 10:33

	Unknown Object (File)
	Sun, Nov 24, 03:44

Subscribers

None

Details

Reviewers

DorianWinty
inidal

Maniphest Tasks

T2027: Export Postman collection to a Bruno repository

Commits

rBRUNO4e1e90d2236c: Import Postman collection for Notifications Delivery API
rBRUNOebf3e0876dae: Import Postman collection for Notifications Delivery API

Summary

Diff Detail

Repository

rBRUNO API collections for Bruno

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

dereckson requested review of this revision.Sep 25 2024, 21:41

dereckson created this revision.

Harbormaster completed remote builds in B5510: Diff 8931.Sep 25 2024, 21:41

Allow filenames to contain spaces

Collections folder are collections names.
As such, filenames can contain spaces.

This is a follow-up for 14472c096b1d.

Harbormaster completed remote builds in B5511: Diff 8932.Sep 25 2024, 21:57

Only Arc linter fix, not the Delivery API collection.

Harbormaster completed remote builds in B5512: Diff 8933.Sep 25 2024, 21:59

dereckson accepted this revision.Sep 25 2024, 21:59

This revision is now accepted and ready to land.Sep 25 2024, 21:59

Closed by commit rBRUNOebf3e0876dae: Import Postman collection for Notifications Delivery API (authored by dereckson). · Explain WhySep 25 2024, 22:00

This revision was automatically updated to reflect the committed changes.

dereckson added a commit: rBRUNOebf3e0876dae: Import Postman collection for Notifications Delivery API.

dereckson reopened this revision.Sep 25 2024, 22:10

This revision is now accepted and ready to land.Sep 25 2024, 22:10

Same player, try again. D3466 was merged by error with this one.

Harbormaster completed remote builds in B5515: Diff 8938.Sep 25 2024, 22:10

dereckson removed a reviewer: dereckson.Sep 25 2024, 22:11

This revision now requires review to proceed.Sep 25 2024, 22:11

All good, just consider using environment variables for the base URL and API keys and you're good to go. Good job! 👍🏻

nasqueron/notifications/Delivery API/get.bru
9	Possible use of environment variables
nasqueron/notifications/Delivery API/register_consumer -404-.bru
9	Possible use of environment variables
20	api-key is hardcoded in 3 files, possible use of environment variable?
nasqueron/notifications/Delivery API/register_consumer -default-.bru
9	Possible use of environment variables
20	api-key is hardcoded in 3 files, possible use of environment variable?
nasqueron/notifications/Delivery API/register_consumer.bru
9	Possible use of environment variables
20	api-key is hardcoded in 3 files, possible use of environment variable?

This revision now requires changes to proceed.Sep 26 2024, 02:19

Indeed, we can save an environment per collection, that makes sense to use it.

Harbormaster completed remote builds in B5518: Diff 8941.Sep 26 2024, 17:29

inidal accepted this revision.Sep 30 2024, 09:59

This revision is now accepted and ready to land.Sep 30 2024, 09:59

Closed by commit rBRUNO4e1e90d2236c: Import Postman collection for Notifications Delivery API (authored by dereckson). · Explain WhyOct 1 2024, 01:02

This revision was automatically updated to reflect the committed changes.

dereckson added a commit: rBRUNO4e1e90d2236c: Import Postman collection for Notifications Delivery API.

Revision Contents
Changeset List

Path

Size

nasqueron/

notifications/

Delivery API/

21 lines

9 lines

environments/

Local development.bru

4 lines

get -base64-.bru

22 lines

get -invalid format-.bru

22 lines

22 lines

register_consumer -404-.bru

22 lines

register_consumer -default-.bru

21 lines

register_consumer.bru

22 lines

unregister_consumer.bru

21 lines

Diff 8960

nasqueron/notifications/Delivery API/200 ALIVE.bru

Loading...

nasqueron/notifications/Delivery API/bruno.json

Loading...

nasqueron/notifications/Delivery API/collection.bru

Loading...

nasqueron/notifications/Delivery API/environments/Local development.bru

Loading...

nasqueron/notifications/Delivery API/get -base64-.bru

Loading...

nasqueron/notifications/Delivery API/get -invalid format-.bru

Loading...

nasqueron/notifications/Delivery API/get.bru

Loading...

nasqueron/notifications/Delivery API/register_consumer -404-.bru

Loading...

nasqueron/notifications/Delivery API/register_consumer -default-.bru

Loading...

nasqueron/notifications/Delivery API/register_consumer.bru

Loading...

nasqueron/notifications/Delivery API/unregister_consumer.bru

Loading...