[Feature Request]: c of teal.slices should treat duplicated `teal_slice` as a shared filters

[gogonzo]

Feature description

IMO when module_specific = TRUE, when two identical slices (not identical(ts), but identical(as.list(<ts>))) then only first one should be used as a "common filter". Ask @gogonzo for more context

c(
  teal::teal_slices(
    teal.slice::teal_slice("ADSL", varname = "AGE", id = "ADSL_AGE"),
    module_specific = TRUE,
    mapping = list(
      `mod1` = "ADSL_AGE"
    )
  ),
    teal::teal_slices(
      teal.slice::teal_slice("ADSL", varname = "AGE", id = "ADSL_AGE"),
      module_specific = TRUE,
      mapping = list(
        `mod2` = "ADSL_AGE"
      )
    )
)
# Error in teal.slice::teal_slices(..., exclude_varnames = exclude_varnames,  : 
#  Some teal_slice objects have the same id:
# ADSL_AGE

Code of Conduct

• I agree to follow this project's Code of Conduct.

Contribution Guidelines

• I agree to follow this project's Contribution Guidelines.

Security Policy

• I agree to follow this project's Security Policy.

[llrs-roche]

Is this an enhancement or a bug fix? From what I understand from the report to me it looks like a bug: the id generated is the same and this prevent both filters to be used(? )

[gogonzo]

@llrs-roche Ok, lets call it a bug. Maybe we were too strict with this limitation in the beginning. I'm taking the issue as it is needed in MDR. I will elaborate more in the PR