feat: add query parameter for activeTab for artifacts and steps

[adong]

Context

Users want to share which active tab they are actively viewing.

Objective

This PR adds a query parameter to preserve what tab is currently active, so that users can share links

Steps: http://localhost:4200/pipelines/1/builds/2

Or
Artifacts: http://localhost:4200/pipelines/1/builds/2?tab=artifacts

References

License

I confirm that this contribution is made under the terms of the license found in the root directory of this repository's source tree and that I have the authority necessary to make this contribution on behalf of its copyright owner.

[tkyi]

Will it be possible to deep link to a specific open artifact? like we deep link to specific steps in build logs

[tkyi]

I think I remember during design discussions that we wanted the activeTab to be "artifacts" if the build had run successfully

[adong]

Oh. I see, that requires some other changes.

[jithine]

This has to be a path parameter, not query param. So url will be http://localhost:4200/pipelines/1/builds/2/artifacts

[adong]

Technical References:

I thought I could setup a query parameter in Controller queryParams, then use setupController in its routers to have a pre-defined default value.

Ran into issues like

1. setting a controller property in setupController does not update the query param emberjs/ember.js#5465
2. https://discuss.emberjs.com/t/how-to-programatically-setup-default-query-params-after-model/13666/7

No success.

Then used Ember.run.next inside Route's setupController, that worked. After read from emberjs/ember.js#14606 (comment)

So I ended up not fighting against the ember framework. Set a query parameter default value in controller, i.e. steps, knowing that default parameter wont show up in the url.

[adong]

Closed in favor of path parameter in #470