Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Trib 263 #103

Closed
wants to merge 13 commits into from
Closed

Conversation

QualityWebDev
Copy link
Contributor

I did the following

  • Imported the actual alert services in ../pages/connect-page/connect-details/connect-details.page.ts.
  • Also in the already mentioned file, I corrected the spacing and indentation.
  • Finally I deleted ../pages/connect-page/_services/alert.

downbythesieh and others added 13 commits March 9, 2024 06:06
…avvato-software#93)

* created connect page service pages

* added appheader, added qr code display

* updated imports, export class, and added qr code methods

* implemented model.service with qr code method

* implemented api.service with qr code method

* added QRCodeModule

* added shared components module and QRCodeModule

* added correct imports to testing modules

* updated package.json with qrcode dependency

* fixed url
…ctions. (savvato-software#95)

* Implemented a call to the backend to retrieve, and then display a list of the current user's connections.

* getting the unit test to pass. required the httpClientTestingModule.
…onnection-details page (savvato-software#97)

* created connection-details pages

* created html with cancel button

* created details.ts page with button navigation

* added link to details page using list connection elements

* updated routing module to reach connection details page

* updated list connections module to include connection details

* created scss page

* changed name of scss page
function to details-page.ts that prints a message in console.
@haxwell
Copy link
Contributor

haxwell commented May 4, 2024

Closing. I think this is duplicate of #100

@haxwell haxwell closed this May 4, 2024
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.

4 participants