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

Add PCCW Parser #173

Open
ryanmerolle opened this issue Oct 12, 2022 · 9 comments
Open

Add PCCW Parser #173

ryanmerolle opened this issue Oct 12, 2022 · 9 comments
Labels
status: help wanted This issue is tentatively accepted pending a volunteer committed to its implementation type: minor feature

Comments

@ryanmerolle
Copy link

Environment

  • circuit_maintenance_parser version: 2.1.0

Proposed Functionality

Add PCCW maintenance parser

Use Case

Useful for clients PCCW.

@chadell
Copy link
Collaborator

chadell commented Oct 13, 2022

Thanks @ryanmerolle
By any chance, do you have an example of a notification from this provider?

@chadell chadell added status: help wanted This issue is tentatively accepted pending a volunteer committed to its implementation type: minor feature labels Oct 13, 2022
@ryanmerolle
Copy link
Author

Of course. Do you need the unaltered email, or can an anonymized PDF work for now?

I did not see any guidelines on submitting the examples.

@chadell
Copy link
Collaborator

chadell commented Oct 13, 2022

There is no guideline to submit examples as usually they come with the parser. You could attach them here, anonymised (but with the original format)

@ryanmerolle
Copy link
Author

I will do so shortly. Thanks!

@snitass
Copy link
Contributor

snitass commented Dec 18, 2024

I would like to add PCCW parser as well. Would you mind if I took over this?

@chadell
Copy link
Collaborator

chadell commented Dec 18, 2024

that would be great!

@snitass
Copy link
Contributor

snitass commented Dec 19, 2024

Their urgent, planned, cancellation and amendment notifications are coming with ics, so I will use SimpleProcessor. The completion notification doesn't come with ics, hence; I will create custom parser for it. Please let me know if I misunderstood anything here.

Thanks

@chadell
Copy link
Collaborator

chadell commented Dec 19, 2024

yes, the idea is to combine different Parsers into one Processor that gets the data from where available.

@snitass
Copy link
Contributor

snitass commented Jan 2, 2025

#301

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: help wanted This issue is tentatively accepted pending a volunteer committed to its implementation type: minor feature
Projects
None yet
Development

No branches or pull requests

3 participants